Skip to content
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.

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

Mapping update processor #32

Merged
merged 25 commits into from
May 17, 2024
Merged

Mapping update processor #32

merged 25 commits into from
May 17, 2024

Conversation

jabberwoc
Copy link
Contributor

@jabberwoc jabberwoc commented Mar 7, 2024

Add processor to run updates of the configured (LOINC) mapping in addition to the existing lab processor.

  • Implement update processing
  • Add Unit tests
  • Update documentation

@jabberwoc jabberwoc added the enhancement New feature or request label Mar 7, 2024
@jabberwoc jabberwoc self-assigned this Mar 7, 2024
Copy link

github-actions bot commented Mar 7, 2024

🦙 MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ ACTION actionlint 3 0 0.07s
✅ COPYPASTE jscpd yes no 2.36s
✅ GROOVY npm-groovy-lint 1 0 12.82s
✅ JAVA checkstyle 28 0 7.97s
✅ MARKDOWN markdownlint 1 0 0.66s
✅ MARKDOWN markdown-link-check 1 0 3.26s
✅ MARKDOWN markdown-table-formatter 1 0 0.58s
✅ REPOSITORY checkov yes no 15.01s
✅ REPOSITORY gitleaks yes no 5.05s
✅ REPOSITORY git_diff yes no 0.01s
✅ REPOSITORY grype yes no 15.63s
✅ REPOSITORY kics yes no 5.85s
✅ REPOSITORY secretlint yes no 1.66s
✅ REPOSITORY trivy yes no 7.45s
✅ REPOSITORY trivy-sbom yes no 5.15s
✅ REPOSITORY trufflehog yes no 11.81s
✅ YAML prettier 6 0 1.26s
✅ YAML v8r 6 0 4.42s
✅ YAML yamllint 6 0 0.38s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by OX Security

Copy link

codecov bot commented Mar 11, 2024

Codecov Report

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

Project coverage is 84.06%. Comparing base (0266aca) to head (4f53281).
Report is 5 commits behind head on main.

Files Patch % Lines
...fhir/configuration/MappingUpdateConfiguration.java 43.54% 30 Missing and 5 partials ⚠️
.../unimarburg/diz/labtofhir/model/MappingUpdate.java 33.33% 16 Missing ⚠️
...rg/diz/labtofhir/processor/LabUpdateProcessor.java 91.11% 1 Missing and 3 partials ⚠️
.../unimarburg/diz/labtofhir/util/ResourceHelper.java 91.66% 1 Missing and 1 partial ⚠️
...in/java/de/unimarburg/diz/labtofhir/LabRunner.java 97.91% 0 Missing and 1 partial ⚠️
...va/de/unimarburg/diz/labtofhir/model/LoincMap.java 96.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main      #32      +/-   ##
============================================
+ Coverage     81.16%   84.06%   +2.90%     
- Complexity      160      227      +67     
============================================
  Files            29       38       +9     
  Lines           807     1092     +285     
  Branches         19       39      +20     
============================================
+ Hits            655      918     +263     
- Misses          129      141      +12     
- Partials         23       33      +10     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jabberwoc jabberwoc merged commit 396e751 into main May 17, 2024
8 checks passed
@jabberwoc jabberwoc deleted the add-mapping-processor branch May 17, 2024 11:00
@jabberwoc
Copy link
Contributor Author

🎉 This PR is included in version 5.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant