New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Dev tools plugin #393

Merged

JPercival merged 13 commits into feature-plugins from DevToolsPlugin

Dec 7, 2021

Contributor

jreyno77 commented Dec 3, 2021

No description provided.

jreyno77 added 7 commits

November 28, 2021 12:32


          Initial Attempt at DevToolsPlugin

0d35b76


          Working through testing, debugging, and refactoring DevPlugin Tests

0960e4e


          Tests passing locally for CodeSystemProviderITs and CacheValueSetsPro…

d5b899d

…viders


          Tests passing locally for CodeSystemProviderITs and CacheValueSetsPro…

e557d1f

…viders


          Merge branch 'DevToolsPlugin' of https://github.com/DBCG/cqf-ruler in…

f7e1a50

…to DevToolsPlugin


          Merge branch 'feature-plugins' of https://github.com/DBCG/cqf-ruler i…

995b060

…nto DevToolsPlugin

Resolved Conflicts:
	pom.xml


          Added tests for Authentication Errors and fixed bugs found from testing

07ccc9f

rob-reynolds self-requested a review

December 4, 2021 00:20

rob-reynolds reviewed

View reviewed changes

rob-reynolds left a comment •

edited

Add plugin to the idea config so that you get those features:
compiler.xml
encodings.xml

rob-reynolds commented Dec 4, 2021

I'd recommend updating the name to dev-tools. "dev" suggests it's for the development environment.

rob-reynolds commented Dec 4, 2021 •

edited

Use the new Description annotation to expose the documentation.
Example: https://github.com/DBCG/cqf-ruler/blob/b160dfc4a2ef3c9da4ebc18df4d94432c85a59a8/plugin/ra/src/main/java/org/opencds/cqf/ruler/plugin/ra/r4/ReportProvider.java#L61

rob-reynolds reviewed

View reviewed changes

plugin/dev/src/main/java/org/opencds/cqf/ruler/plugin/dev/utils/DevUtilities.java Outdated Show resolved Hide resolved

rob-reynolds reviewed

View reviewed changes

plugin/dev/src/main/java/org/opencds/cqf/ruler/plugin/dev/utils/IDevUtilities.java Outdated Show resolved Hide resolved

rob-reynolds reviewed

View reviewed changes

plugin/dev/src/main/resources/application.yaml Outdated Show resolved Hide resolved

rob-reynolds reviewed

View reviewed changes

plugin/dev/src/test/java/org/opencds/cqf/ruler/plugin/dev/IServerSupport.java Outdated

		@@ -0,0 +1,60 @@
		package org.opencds.cqf.ruler.plugin.dev;

rob-reynolds Dec 4, 2021

Use / move this to the utility plugin

rob-reynolds reviewed

View reviewed changes

plugin/dev/src/test/java/org/opencds/cqf/ruler/plugin/dev/TesterUtilities.java Outdated Show resolved Hide resolved

rob-reynolds reviewed

View reviewed changes

plugin/dev/src/test/java/org/opencds/cqf/ruler/plugin/dev/dstu3/CacheValueSetsProviderIT.java Outdated Show resolved Hide resolved

rob-reynolds requested changes

View reviewed changes

rob-reynolds left a comment

See all the comments

rob-reynolds commented Dec 4, 2021

Compiles locally.
Some tests did not pass locally.

rob-reynolds and others added 6 commits

December 3, 2021 18:19


          Auto format imports on save

a888e71


          Removed unnecessary Utility Classes

821ff0b

Added Hapi Description Annotations to Operations
Removed unnecessary application properties
added dev-tools package to compiler.xml and encodings.xml


          Rename plugin module to dev-tools and package to devtools

b254e16


          Merge branch 'feature-plugins' into DevToolsPlugin

c5861de


          Updates for tests

06ca8c5


          add debug

b53943d

JPercival approved these changes

View reviewed changes

JPercival merged commit cbff28a into feature-plugins

JPercival deleted the DevToolsPlugin branch

December 7, 2021 17:11

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment