cqframework / cqf-tooling

Tooling for CQL and IG Authors
Apache License 2.0
19 stars 22 forks source link

Prep for 3.2.0 release #517

Closed JPercival closed 5 months ago

JPercival commented 5 months ago

Description

By creating this PR you acknowledge that your contribution will be licensed under Apache 2.0

ddieppois commented 5 months ago

@JPercival I noticed that the build was failing due to a conflict with Wiremock dependency which contained multiple version of Jetty (11 and 12). I noticed that multiple new version of wiremock came out with less vulnerability so I decided to update to the latest version and force the use of jetty 11.0.20 (as per wiremock requirement) with a dependency management node. This seems to have fixed the issue and the only test that was failing du to Wiremock is now working.

JPercival commented 5 months ago

looks good to me ! I'll make sure to put the coming test results into target/test-output!

When you run mvn clean, it automatically deletes the target folder. It's also ignored in git. This way no test output gets accidentally committed and it's self-cleaning.

codecov[bot] commented 5 months ago

Codecov Report

Attention: Patch coverage is 12.50000% with 14 lines in your changes are missing coverage. Please review.

Project coverage is 20.85%. Comparing base (241bd02) to head (e8a2b59).

Files Patch % Lines
...g/cql_generation/builder/VmrToModelElmBuilder.java 0.00% 2 Missing :warning:
...opencds/cqf/tooling/processor/IGTestProcessor.java 0.00% 2 Missing :warning:
...cds/cqf/tooling/terminology/SpreadsheetHelper.java 50.00% 2 Missing :warning:
...org/opencds/cqf/tooling/utilities/R4FHIRUtils.java 0.00% 2 Missing :warning:
...g/opencds/cqf/tooling/utilities/STU3FHIRUtils.java 0.00% 2 Missing :warning:
...rkit/StructureDefinitionElementBindingVisitor.java 0.00% 1 Missing :warning:
...eleratorkit/StructureDefinitionElementVisitor.java 0.00% 1 Missing :warning:
.../opencds/cqf/tooling/quick/QuickPageGenerator.java 0.00% 1 Missing :warning:
...cqf/tooling/terminology/VSACValueSetGenerator.java 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #517 +/- ## ============================================= + Coverage 0 20.85% +20.85% - Complexity 0 1494 +1494 ============================================= Files 0 282 +282 Lines 0 24090 +24090 Branches 0 3832 +3832 ============================================= + Hits 0 5024 +5024 - Misses 0 18221 +18221 - Partials 0 845 +845 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.