Skip to content

[codex] Fix matrix-arff release findings#298

Merged
perNyfelt merged 5 commits intomainfrom
matrix-arff-0.2.1
Apr 30, 2026
Merged

[codex] Fix matrix-arff release findings#298
perNyfelt merged 5 commits intomainfrom
matrix-arff-0.2.1

Conversation

@perNyfelt
Copy link
Copy Markdown
Member

Summary

  • Quote ARFF-sensitive nominal literals so values such as ?, empty strings, and %... round-trip as data instead of missing values or comments.
  • Add shared strict DATE formatter handling for ARFF parsing and writing using Locale.ROOT and UTC.
  • Fix the matrix-arff published SCM URL to point at the module on main.
  • Add regression coverage for nominal sentinel round trips, invalid date rejection, and UTC date output.

Impact

This avoids lossy ARFF nominal serialization and makes DATE parsing fail on invalid dates instead of silently normalizing them. The POM metadata now points to the correct module location for release consumers.

Validation

  • ./gradlew :matrix-arff:test
  • ./gradlew :matrix-arff:build
  • git diff --check

@perNyfelt perNyfelt marked this pull request as ready for review April 30, 2026 21:18
@perNyfelt perNyfelt merged commit c6edef9 into main Apr 30, 2026
1 check passed
@perNyfelt perNyfelt deleted the matrix-arff-0.2.1 branch April 30, 2026 21:19
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.

1 participant