Skip to content

Restore serialization to use interfaces again#1834

Merged
uhafner merged 1 commit intomainfrom
loose-coupling
Apr 14, 2026
Merged

Restore serialization to use interfaces again#1834
uhafner merged 1 commit intomainfrom
loose-coupling

Conversation

@uhafner
Copy link
Copy Markdown
Owner

@uhafner uhafner commented Apr 14, 2026

Changing the fields to serializable classes breaks the existing serializations.

@uhafner uhafner added the bug Bugs or performance problems label Apr 14, 2026
Changing the fields to serializable classes breaks the existing
serializations.
@github-actions
Copy link
Copy Markdown

☀️   Quality Monitor

Tests

   JUnit   Unit Tests: ✅ successful — 87 passed, 2 skipped
   ⛔   Architecture Tests: ✅ successful — 17 passed, 2 skipped

Coverage for New Code

   〰️   Line Coverage: 100.00% — perfect 🎉
   ➰   Branch Coverage: 100.00% — perfect 🎉
   PIT   Mutation Coverage: 100.00% — perfect 🎉
   💪   Test Strength: 100.00% — perfect 🎉

Coverage for Whole Project

   〰️   Line Coverage: 91.57% — 65 missed lines
   ➰   Branch Coverage: 90.45% — 21 missed branches
   PIT   Mutation Coverage: 91.61% — 26 survived mutations
   💪   Test Strength: 94.67% — 16 survived mutations in tested code

Style

   CheckStyle   CheckStyle: No warnings
   PMD   PMD: No warnings
   ☕   Java Compiler: No warnings

Bugs

   SpotBugs   SpotBugs: No bugs
   🐛   Error Prone: No bugs

API Problems

   🚫   Revapi: No warnings

Vulnerabilities

   🛡️   OWASP Dependency Check: No vulnerabilities

Software Metrics

   🌀   Cyclomatic Complexity: 384 (total)
   💭   Cognitive Complexity: 174 (total)
   ➿   N-Path Complexity: 348 (total)
   📏   Lines of Code: 4412 (total)
   📝   Non Commenting Source Statements: 1274 (total)
   🔗   Class Cohesion: 71.43% (maximum)
   ⚖️   Weight of Class: 100.00% (maximum)

🚦 Quality Gates

Overall Status: ✅ SUCCESS

✅ Passed Gates

  • ✅ Overall Tests Success Rate: 100.00 >= 100.00
  • ✅ Line Coverage in New Code: 100.00 >= 90.00
  • ✅ Branch Coverage in New Code: 100.00 >= 90.00
  • ✅ Mutation Coverage in New Code: 100.00 >= 90.00
  • ✅ Potential Bugs in Whole Project: 0.00 <= 0.00
  • ✅ Style Violation in Whole Project: 0.00 <= 0.00

Created by Quality Monitor v4.10.0 (#be1b382). More details are shown in the GitHub Checks Result.

@uhafner uhafner merged commit d655a32 into main Apr 14, 2026
16 checks passed
@uhafner uhafner deleted the loose-coupling branch April 14, 2026 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Bugs or performance problems

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant