-
Notifications
You must be signed in to change notification settings - Fork 9
Fix hl7 extensions inconsistencies #508
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
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #508 +/- ##
==========================================
+ Coverage 25.06% 25.51% +0.44%
==========================================
Files 18 18
Lines 2541 2559 +18
==========================================
+ Hits 637 653 +16
- Misses 1904 1906 +2 ☔ View full report in Codecov by Sentry. |
Stole this from IPM Some more work needed still
This code does not actually fix the issue
Also skip the directory scan because it'd be slow and is generally unnecessary.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed this with @isc-etamarch in a few calls today. There are a few other things we'll want to do but this is good for purposes of the current issue/PR.
@isc-pbarton I'd appreciate if you could review too since I made a few of the changes here.
Addresses #495