Skip to content

Conversation

@palukku
Copy link
Member

@palukku palukku commented Sep 9, 2025

Closes #13746

Added the integrity check to jabkit.

Steps to test

You can test the different formats with the following commands:

.\gradlew.bat jabkit:run --args="check-integrity --output-format=csv --input=../jablib/src/main/resources/Chocolate.bib --porcelain"
.\gradlew.bat jabkit:run --args="check-integrity --output-format=txt --input=../jablib/src/main/resources/Chocolate.bib"
.\gradlew.bat jabkit:run --args="check-integrity --output-format=errorformat --input=../jablib/src/main/resources/Chocolate.bib"

Mandatory checks

@palukku
Copy link
Member Author

palukku commented Sep 9, 2025

Openrewrite misinterprets the NotificationService "notify" as an Object#notify...

image image

@palukku palukku requested a review from koppor September 10, 2025 19:39
calixtus
calixtus previously approved these changes Sep 12, 2025
@JabRef JabRef deleted a comment from trag-bot bot Sep 12, 2025
@calixtus calixtus enabled auto-merge September 12, 2025 08:32
Siedlerchr
Siedlerchr previously approved these changes Sep 12, 2025
@palukku palukku dismissed stale reviews from Siedlerchr and calixtus via 4bd013e September 12, 2025 08:59
@trag-bot
Copy link

trag-bot bot commented Sep 12, 2025

@trag-bot didn't find any issues in the code! ✅✨

@calixtus calixtus added this pull request to the merge queue Sep 12, 2025
Merged via the queue into JabRef:main with commit 5fa313d Sep 12, 2025
37 checks passed
Siedlerchr added a commit that referenced this pull request Sep 12, 2025
* upstream/main:
  Add integrity check to cli (#13848)
Siedlerchr added a commit that referenced this pull request Sep 13, 2025
* upstream/main:
  New Crowdin updates (#13865)
  Fix unresponsive "Check Consistency" dialog (#13864)
  Fix several aux group issues (#13861)
  Add integrity check to cli (#13848)
  Add sbom file and remove external libraries (#13859)
  Fix Tagsfield in Keyword editor (#13858)
  Chore(deps): Bump jablib/src/main/resources/csl-styles (#13857)
  Chore(deps): Bump jablib/src/main/resources/csl-locales (#13856)
@palukku palukku changed the title Add integrity check to cli Feat: Add integrity check to cli Nov 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add "check-integrity" to CLI

4 participants