-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: Fix Ordering issue in PBJ Comparable handling #189
base: main
Are you sure you want to change the base?
Conversation
…ration tests Signed-off-by: Joseph Sinclair <[email protected]>
5314bd1
to
b4de41a
Compare
Coverage summary from CodacySee diff coverage on Codacy
Coverage variation details
Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: Diff coverage details
Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: See your quality gate settings Change summary preferencesYou may notice some variations in coverage metrics with the latest Coverage engine update. For more details, visit the documentation |
bfa8b1a
to
f2f0559
Compare
f2f0559
to
0dc8a78
Compare
* Added messages to test protobufs that fail in CI * Adjusted getJavaFile to check existence and provide clearer error messages if a required file is missing * Moved a few strings to constants in Common.java * Removed a few unnecessary `toString` calls * Fixed wildcard static import in Field.java --- Additional minor adjustments * Updated the Java language version to 21. * Updated github CI config to use Java 21 * Moved the folder for the protobuf grammar file (Protobuf3.g4) to correct a package mismatch Signed-off-by: Joseph Sinclair <[email protected]>
0dc8a78
to
aff86a9
Compare
toString
callsAdditional minor adjustments