Fix for JabKit CLI help output format #13275 #14440
Merged
+32
−8
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…at support Extract option checking logic into helper method and use Map to track command options. Add constants for footer labels and implement support for export format footers alongside existing input and output formats.
Move hasCommandOption method below applyUsageFooters for better logical grouping and add comprehensive JavaDoc documentation to explain the usage footer application functionality.
Contributor
Hey @opazenha!Thank you for contributing to JabRef! Your help is truly appreciated ❤️ We have automated checks in place, based on which you will soon get feedback if any of them are failing. In a while, maintainers will also review your contribution. Once that happens, you can go through their comments in the "Files changed" tab and act on them, or reply to the conversation if you have further inputs. Please re-check our contribution guide in case of any other doubts related to our contribution workflow. |
opazenha
force-pushed
the
fix-for-issue-13275
branch
from
b2ebb8b to
057e0fd
Compare
- Replace hardcoded strings with Localization.lang() calls - Add missing 'Available output formats:' localization key - Fixes failing LocalizationConsistencyTest
opazenha
changed the title
koppor
added
the
status: ready-for-review
calixtus
approved these changes
Nov 26, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #13275
Refactored jabkit CLI to improve output formatting and code organization. Changes include clearing unused imports, skipping redundant format footers for check-consistency command, reorganizing method order, adding documentation, and improving usage footer generation with export format support.
Steps to test
./gradlew build./gradlew :jabkit-cli:run --args="check-consistency"./gradlew :jabkit-cli:run --args="export --help"Mandatory checks
CHANGELOG.mdin a way that is understandable for the average user (if change is visible to the user)