Skip to content

Comments

Update KIDL code to the most recent kb_sdk_plus version.#7

Merged
MrCreosote merged 2 commits intodevelopfrom
dev-update_kidl
Mar 31, 2025
Merged

Update KIDL code to the most recent kb_sdk_plus version.#7
MrCreosote merged 2 commits intodevelopfrom
dev-update_kidl

Conversation

@MrCreosote
Copy link
Member

KIDL code should next be removed from kb_sdk_plus and a jar from this repo used instead.

KIDL code should next be removed from kb_sdk_plus and a jar from this
repo used instead.
@MrCreosote MrCreosote requested a review from briehl March 27, 2025 17:08
@codecov
Copy link

codecov bot commented Mar 27, 2025

Codecov Report

Attention: Patch coverage is 58.83495% with 212 lines in your changes missing coverage. Please review.

Project coverage is 62.14%. Comparing base (569cf23) to head (2faa21f).
Report is 3 commits behind head on develop.

Files with missing lines Patch % Lines
src/main/java/us/kbase/jkidl/SpecParser.java 67.60% 20 Missing and 3 partials ⚠️
src/main/java/us/kbase/kidl/KbAnnotations.java 73.17% 20 Missing and 2 partials ⚠️
src/main/java/us/kbase/kidl/KbFuncdef.java 41.17% 19 Missing and 1 partial ⚠️
src/main/java/us/kbase/kidl/KbModule.java 55.55% 19 Missing and 1 partial ⚠️
src/main/java/us/kbase/kidl/KbTypedef.java 4.76% 19 Missing and 1 partial ⚠️
src/main/java/us/kbase/kidl/KbTuple.java 20.83% 19 Missing ⚠️
src/main/java/us/kbase/kidl/KbStruct.java 22.22% 14 Missing ⚠️
src/main/java/us/kbase/kidl/KbScalar.java 7.14% 13 Missing ⚠️
src/main/java/us/kbase/kidl/JSONableVisitor.java 88.54% 5 Missing and 6 partials ⚠️
src/main/java/us/kbase/kidl/KbParameter.java 40.00% 9 Missing ⚠️
... and 8 more
Additional details and impacted files
@@              Coverage Diff              @@
##             develop       #7      +/-   ##
=============================================
+ Coverage      61.50%   62.14%   +0.63%     
- Complexity       634      689      +55     
=============================================
  Files             35       37       +2     
  Lines           2816     2953     +137     
  Branches         554      539      -15     
=============================================
+ Hits            1732     1835     +103     
- Misses           897      931      +34     
  Partials         187      187              

☔ 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.

Comment on lines -5 to -6
* Similarly, the HTML generation code should be moved into the KIDL code, perhaps in
`kidlhtml` or `kidl.html`.
Copy link
Member Author

Choose a reason for hiding this comment

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

Changed my mind here, if anything it should be moved to its own package, but for now leaving it in the SDK is fine

Copy link
Member

@briehl briehl left a comment

Choose a reason for hiding this comment

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

👍

@MrCreosote MrCreosote merged commit 5507412 into develop Mar 31, 2025
6 of 7 checks passed
@MrCreosote MrCreosote deleted the dev-update_kidl branch March 31, 2025 20:56
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.

2 participants