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

SO-6321: DSV simple type export should consider the characteristic type of properties #1349

Merged
merged 6 commits into from
Nov 28, 2024

Conversation

apeteri
Copy link
Member

@apeteri apeteri commented Nov 27, 2024

No description provided.

...of "additional" in non-zero groups only in the simple type DSV refset
exporter along with all inferred ones; add small test cases to
demonstrate behavior
@apeteri apeteri self-assigned this Nov 27, 2024
Copy link

codecov bot commented Nov 27, 2024

Codecov Report

Attention: Patch coverage is 92.94118% with 12 lines in your changes missing coverage. Please review.

Project coverage is 64.10%. Comparing base (c783a75) to head (8701276).
Report is 10 commits behind head on 9.x.

Files with missing lines Patch % Lines
...request/dsv/SnomedSimpleTypeRefSetDSVExporter.java 92.94% 5 Missing and 7 partials ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##                9.x    #1349      +/-   ##
============================================
+ Coverage     63.69%   64.10%   +0.41%     
- Complexity    12535    12616      +81     
============================================
  Files          1613     1614       +1     
  Lines         57218    57266      +48     
  Branches       5629     5633       +4     
============================================
+ Hits          36445    36711     +266     
+ Misses        18421    18197     -224     
- Partials       2352     2358       +6     

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

...concept term" column, otherwise no literals will be added to exported
DSVs that don't have the destination SCTID column enabled. Also add test
cases that exercise relationship export items.
@apeteri apeteri requested review from cmark and AAAlinaaa November 27, 2024 22:34
@apeteri apeteri marked this pull request as ready for review November 27, 2024 22:34
Copy link
Member

@cmark cmark left a comment

Choose a reason for hiding this comment

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

🥓

@cmark cmark merged commit 0237627 into 9.x Nov 28, 2024
5 checks passed
@cmark cmark deleted the issue/SO-6321-DSV-export-items branch November 28, 2024 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants