Skip to content

Conversation

yangxk1
Copy link
Contributor

@yangxk1 yangxk1 commented Sep 29, 2025

Reason for this PR

close #778

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 70.96774% with 18 lines in your changes missing coverage. Please review.
✅ Project coverage is 76.68%. Comparing base (6f112c6) to head (2f66dde).

Files with missing lines Patch % Lines
...java/org/apache/graphar/info/type/Cardinality.java 62.50% 3 Missing and 3 partials ⚠️
.../apache/graphar/info/saver/BaseGraphInfoSaver.java 37.50% 3 Missing and 2 partials ⚠️
...rc/main/java/org/apache/graphar/info/EdgeInfo.java 50.00% 1 Missing and 1 partial ⚠️
...ava/org/apache/graphar/info/yaml/PropertyYaml.java 75.00% 1 Missing and 1 partial ⚠️
.../java/org/apache/graphar/info/yaml/VertexYaml.java 66.66% 1 Missing and 1 partial ⚠️
...rc/main/java/org/apache/graphar/info/Property.java 90.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main     #780      +/-   ##
============================================
+ Coverage     76.59%   76.68%   +0.08%     
- Complexity      560      583      +23     
============================================
  Files            83       84       +1     
  Lines          8679     8736      +57     
  Branches       1006     1012       +6     
============================================
+ Hits           6648     6699      +51     
- Misses         1813     1814       +1     
- Partials        218      223       +5     
Flag Coverage Δ
java-info 83.09% <70.96%> (+0.34%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@yangxk1 yangxk1 requested a review from Thespica September 29, 2025 07:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat(java,info): support multi-property in yaml #768
2 participants