Skip to content

Restore serialization to use interfaces again#728

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

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

Conversation

@uhafner
Copy link
Copy Markdown
Member

@uhafner uhafner commented Apr 14, 2026

Changing the fields to serializable classes breaks the existing serializations.

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

☀️   Quality Monitor

Tests

   JUnit   Unit Tests: ✅ successful — 190 passed
   🚀   Integration Tests: ✅ successful — 148 passed, 7 skipped
   ⛔   Architecture Tests: ✅ successful — 15 passed

Coverage for New Code

   〰️   Line Coverage: 100.00% — perfect 🎉
   ➰   Branch Coverage: 100.00% — perfect 🎉

Coverage for Whole Project

   〰️   Line Coverage: 76.24% — 727 missed lines
   ➰   Branch Coverage: 66.63% — 292 missed branches

Style

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

Bugs

   SpotBugs   SpotBugs: No bugs
   🐛   Error Prone: No bugs

Vulnerabilities

   🛡️   OWASP Dependency Check: 206 vulnerabilities — error: 18, high: 53, normal: 129, low: 6

Software Metrics

   🌀   Cyclomatic Complexity: 1028 (total)
   💭   Cognitive Complexity: 562 (total)
   ➿   N-Path Complexity: 1385 (total)
   📏   Lines of Code: 12044 (total)
   📝   Non Commenting Source Statements: 4058 (total)
   🔗   Class Cohesion: 100.00% (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
  • ✅ 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 5003345 into main Apr 14, 2026
35 checks passed
@uhafner uhafner deleted the loose-coupling branch April 14, 2026 12:44
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