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

Platform/david navapbc/15514/hl7 oml aui fhir mapping #16619

Merged
merged 22 commits into from
Dec 5, 2024

Conversation

david-navapbc
Copy link
Collaborator

This PR ...

adds mapping for HL7/IN1/AUI to HL7/OML and AUIExtension test

Testing

  • [ x] Tested locally?
  • [ x] Ran ./prime test or ./gradlew testSmoke against local Docker ReportStream container?
  • [ x] Added tests?

@david-navapbc david-navapbc requested a review from a team as a code owner November 20, 2024 23:36
Copy link
Contributor

github-actions bot commented Nov 20, 2024

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Manifest Files

Copy link
Contributor

github-actions bot commented Nov 20, 2024

Test Results

1 255 tests  ±0   1 251 ✅ ±0   7m 49s ⏱️ +5s
  164 suites ±0       4 💤 ±0 
  164 files   ±0       0 ❌ ±0 

Results for commit 912c372. ± Comparison against base commit 66c3691.

♻️ This comment has been updated with latest results.

Copy link
Contributor

github-actions bot commented Nov 21, 2024

Integration Test Results

 59 files   59 suites   37m 52s ⏱️
423 tests 413 ✅ 10 💤 0 ❌
426 runs  416 ✅ 10 💤 0 ❌

Results for commit 912c372.

♻️ This comment has been updated with latest results.

@JFisk42 JFisk42 self-assigned this Nov 21, 2024
@JFisk42 JFisk42 added the platform Platform Team label Nov 21, 2024
Copy link
Collaborator

@JFisk42 JFisk42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few comments relating to formatting/project convention following, otherwise this was great work!

Copy link
Collaborator

@JFisk42 JFisk42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Noticed a few small issues from merging main in that my day-before-thanksgiving brain missed last week.

Copy link

sonarqubecloud bot commented Dec 5, 2024

@JFisk42 JFisk42 merged commit e564319 into main Dec 5, 2024
25 checks passed
@JFisk42 JFisk42 deleted the platform/david-navapbc/15514/hl7_oml_aui_fhir_mapping branch December 5, 2024 20:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
platform Platform Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants