Skip to content

Conversation

@marhali
Copy link
Owner

@marhali marhali commented Sep 16, 2025

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Sep 20, 2025

Codecov Report

❌ Patch coverage is 74.84076% with 237 lines in your changes missing coverage. Please review.
✅ Project coverage is 72.31%. Comparing base (51db083) to head (a5ac1a0).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
.../main/java/de/marhali/json5/stream/Json5Lexer.java 50.34% 83 Missing and 63 partials ⚠️
src/main/java/de/marhali/json5/Json5Primitive.java 78.18% 13 Missing and 11 partials ⚠️
...main/java/de/marhali/json5/stream/Json5Writer.java 85.29% 10 Missing and 10 partials ⚠️
.../java/de/marhali/json5/internal/LinkedTreeMap.java 59.52% 9 Missing and 8 partials ⚠️
...ain/java/de/marhali/json5/config/Json5Options.java 90.67% 11 Missing ⚠️
src/main/java/de/marhali/json5/Json5Array.java 88.37% 2 Missing and 3 partials ⚠️
...main/java/de/marhali/json5/stream/Json5Parser.java 90.38% 1 Missing and 4 partials ⚠️
src/main/java/de/marhali/json5/Json5Object.java 80.00% 0 Missing and 4 partials ⚠️
...e/marhali/json5/internal/EcmaScriptIdentifier.java 95.83% 0 Missing and 2 partials ⚠️
...in/java/de/marhali/json5/internal/RadixNumber.java 81.81% 0 Missing and 2 partials ⚠️
... and 1 more
Additional details and impacted files
@@             Coverage Diff              @@
##               main      #29      +/-   ##
============================================
+ Coverage     67.62%   72.31%   +4.68%     
- Complexity      297      534     +237     
============================================
  Files            18       19       +1     
  Lines           970     1575     +605     
  Branches        170      316     +146     
============================================
+ Hits            656     1139     +483     
- Misses          218      286      +68     
- Partials         96      150      +54     

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

@marhali marhali force-pushed the develop branch 2 times, most recently from 5d2efa4 to f9291ea Compare September 23, 2025 16:31
@github-actions
Copy link

github-actions bot commented Sep 23, 2025

Qodana Community for JVM

21 new problems were found

Inspection name Severity Problems
Dangling Javadoc comment 🔶 Warning 8
Nullability and data flow problems 🔶 Warning 4
String concatenation in loop 🔶 Warning 2
Call to 'BigDecimal' method without a rounding mode argument 🔶 Warning 1
Constant values 🔶 Warning 1
'equals()' method that does not check the class of its parameter 🔶 Warning 1
String comparison using '==', instead of 'equals()' 🔶 Warning 1
Redundant 'if' statement 🔶 Warning 1
Unused assignment 🔶 Warning 1
Constant values ◽️ Notice 1

☁️ View the detailed Qodana report

Contact Qodana team

Contact us at qodana-support@jetbrains.com

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants