[2025-04-10T13:03:05.686Z] code review started
[2025-04-10T13:03:05.686Z] owner: JabRef
[2025-04-10T13:03:05.686Z] repo: jabref
[2025-04-10T13:03:05.686Z] repoUrl: https://github.com/JabRef/jabref
[2025-04-10T13:03:05.686Z] author: priyanshu16095
[2025-04-10T13:03:05.686Z] listing pull request files
[2025-04-10T13:03:15.992Z] total file count: 30
[2025-04-10T13:03:15.992Z] eligible file count: 28
[2025-04-10T13:03:16.045Z] pro user privilege applied
[2025-04-10T13:03:16.045Z] getting project rules
[2025-04-10T13:03:16.057Z] model: claude-3-5-sonnet-20240620
[2025-04-10T13:03:16.057Z] on rule review mode: true
[2025-04-10T13:03:16.057Z] glob ignore:
[2025-04-10T13:03:16.057Z] pull number: 12495
[2025-04-10T13:03:16.057Z] projectId: 885262de-6ea4-406f-ab33-0a18e077aaf9
[2025-04-10T13:03:18.192Z] Found 44 existing review comments
[2025-04-10T13:03:18.193Z] file: CHANGELOG.md
[2025-04-10T13:03:18.193Z] file is outside of new changes on pr, skipping
[2025-04-10T13:03:18.193Z] file: src/main/java/org/jabref/gui/JabRefGUI.java
[2025-04-10T13:03:18.193Z] file is outside of new changes on pr, skipping
[2025-04-10T13:03:18.193Z] file: src/main/java/org/jabref/gui/integrity/IntegrityCheckDialog.fxml
[2025-04-10T13:03:18.193Z] not a code file, skipping
[2025-04-10T13:03:18.193Z] file: src/main/java/org/jabref/gui/integrity/IntegrityCheckDialog.java
[2025-04-10T13:03:18.193Z] reading file blob
[2025-04-10T13:03:20.985Z] filteredRules: 1. If a method has JavaDoc and code of the method has changed, the JavaDoc has to be updated accordingly. No need to add JavaDoc for "trivial" exceptions
2. If code in org.jabref.model or org.jabref.logic has been changed, tests need to be adapted or updated accordingly
3. The code should follow the fail fast principle by immediately handling invalid states and returning early instead of nesting logic inside else branches.

Example:

Bad:

if (path.isEmpty()) {
return false;
} else {
// other code
}

Good:

if (path.isEmpty()) {
return false;
}

// other code
4. The pull request title should contain a short title of the issue fixed (or what the PR adresses) and not just "Fix issue xyz"
5. The "Mandatory checks" are Markdown TODOs. They should be formatted as that. Wrong: - [ x]. Either - [ ] or - [x].
6. New methods (and new classes) should follow the Single-responsibility principle (SRP).
7. There should be JavaDoc for complex methods.
8. "Magic" numbers or strings should be constants - or at least have a Java comment. Exception: JavaFX height and widths.
9. Avoid code duplication
10. Use modern Java best practices, such as Arguments.of() instead of new Object[] especially in JUnit tests or Path.of() instead of Paths.get(), to improve readability and maintainability.
11. Exceptions should be used for exceptional states - not for normal control flow
12. Follow the principles of "Effective Java"
13. In JabRef, localized strings are done using Localization.lang("string"). More information at https://devdocs.jabref.org/code-howtos/localization.html.
14. No use of Java SWING, only JavaFX is allowed as UI technology
15. @DisplayName for tests should only be used if absolutely necessary: The method name itself should be comprehensive enough.
16. Comments should add new information (e.g. reasoning of the code). It should not be plainly derived from the code itself.

Example for trivail comments:

    // Commit the staged changes
    RevCommit commit = git.commit()

    fieldName = fieldName.trim().toLowerCase(); // Trim and convert to lower case

17. Instead of Files.createTempDirectory @TempDir JUnit5 annotation should be used.
18. Do not catch the general java java.lang.Exception. Catch specific exeptions only
19. Avoid exclamation marks at the end of a sentence. They are more for screaming. Use a dot to end the sentence.
20. All labels and texts should be sentence case (and not title case)
21. New public methods should not return null. They should make use of java.util.Optional. In case null really needs to be used, the JSpecify annotations must be used.
22. Use "BibTeX" as spelling for bibtex in Java strings. In variable names "Bibtex" should be used.
23. New strings should be consistent to other strings. They should also be grouped semantically together.
24. Existings strings should be reused instead of introducing slightly different strings
25. Comments on/above methods should be JavaDoc.not simple Java comments //. Three /// are OK (because this is Java23 and later)
26. The CHANGELOG.md entry should be for end users (and not programmers).
27. User dialogs should have proper button labels: NOT yes/no/cancel, but indicating the action which happens when pressing the button
28. GUI code should only be a gateway to code in org.jabref.logic. More complex code regarding non-GUI operations should go into org.jabref.logic. Think of layerd archicture.
29. null should never be passed to a method (except it has the same name).
30. Do not add extra blank lines in CHANGELOG.md
31. Remove commented code. (To keep a history of changes git was made for.)
32. Do not use Objects.requireNonNull. Use JSpecify @NonNull annotation if needs be.
33. Do not throw unchecked exceptions (e.g., do not throw new RuntimeException, do not throw new IllegalStateException)

Reason: This tears down the whole application. One does not want to loose data only because "a corner" of the application broke.
34. When adding JavaDoc, the text should be non-trivial.

Example for trivial JavaDoc:

 * @param backupDir the backup directory
 * @param dbfile the database file
 * @throws IOException     if an I/O error occurs
 * @throws GitAPIException if a Git API error occurs

35. Code should not be reformatted only because of syntax. There need to be new statements added if reformatting.
36. If @TempDir is used, there is no need to clean it up

Example for wrong code:

    @AfterEach
    void tearDown() throws IOException {
        FileUtils.cleanDirectory(tempDir.toFile());
    }

37. Assert the contents of objects (assertEquals), not checking for some Boolean conditions (assertTrue/assertFalse)

Example for wrong code:

        assertTrue(
                entry.getFiles().stream()
                     .anyMatch(file -> file.getLink().equals(newFile.getFileName().toString()) ||
                             file.getLink().endsWith("/" + newFile.getFileName().toString()))
        );

38. No "new Thread()", use "org.jabref.logic.util.BackgroundTask" and its "executeWith"
39. try blocks shoud cover as less statements as possible (and not whole methods)
40. use "throws Exception" in the method declaration instead of try-catch and fail/log/...
41. When creating a new BibEntry object "withers" should be used: Instead of setField, withField methods should be used.
42. In case Java comments are added, they should match the code following. They should be a high-level summary or guidance of the following code (and not some reandom text or just re-stating the obvious)
43. Use the methods of java.util.Optional. ifPresent.

NOT

Optional<String> resolved = bibEntry.getResolvedFieldOrAlias(...);
String value = resolved.orElse("");
doSomething(value)

Following is fine:

Optional<String> resolved = bibEntry.getResolvedFieldOrAlias(...);
resolved.ifPresent(value -> doSomething(value));

44. If the java.util.Optional is really present, use use get() (and not orElse(""))
45. Use Java Text blocks (""") for multiline string constants
46. Use compiled patterns (Pattern.compile)

Examples:

NOT: x.matches(".\s{2,}.")

BUT:
private final static PATTERN = ...
and then PATTERN.matcher(x)
47. Use placeholders if variance is in localizaiton:

BAD: Localization.lang("Current JabRef version") + ": " + buildInfo.version);

GOOD: Localization.lang("Current JabRef version: %0", buildInfo.version);
48. Log exceptions using exception logging capabilities

BAD: LOGGER.info("Failed to push: ".concat(e.toString()));

GOOD: LOGGER.info("Failed to push", e);
49. Boolean method parameters (for public methods) should be avoided. Better create two distinct methods (which maybe call some private methods)
50. Use modern Java data structures

BAD: new HashSet<>(Arrays.asList(...))

GOOD: Sef.of(...)
51. Use logger conversion to log objects - do not do manual conversion. Especially, java.util.Optional can be logged directly, no need for get(), ,getOrElse(), ...

BAD:

        LOGGER.info(String.valueOf(path.get()));
        LOGGER.info("Not a git repository");

GOOD:

       LOGGER.info("Not a git repository", path);

52. Java 21 introduced SequencedCollection and SequencedSet interfaces. Use it instead of LinkedHashSet (where applicable)
53. Use StringJoiner instead of StringBuilder (if possible)
54. In case defaults are modified, there needs to be a preference migration.
55. Proper localization. In JabRef, localized strings are prefix with %

BAD: <Label ... text="Git"/>

GOOD: <Label ... text="%Git"/>
56. Labels should not end with ":"

BAD:

GOOD:
57. Plain JUnit assert should be used instead of org.assertj (if possible)

BAD: assertThat(gitPreferences.getAutoPushEnabled()).isFalse();

GOOD: assertFalse(gitPreferences.getAutoPushEnabled());
58. Do not catch exceptions in Test - let JUnit handle

BAD: try {...code...} catch (IOException e) {
throw new AssertionError("Failed to set up test directory", e);
}

GOOD: ...code...
59. Name variables using camelcase (and not snake_case, etc)
60. Minimal quality for variable names: Not extraEntry2, extraEntry3; but include meaning/intention into the variable names
61. One should use jabref's dialogService (instead of Java native FileChooser)

dialogService.showFileOpenDialog(fileDialogConfiguration).ifPresent(path -> ...)

and with FileDialogConfiguration offers the Builder pattern.
(see e.g NewLibraryFromPdfAction)

[2025-04-10T13:03:47.185Z] filtering out non-relevant issues
[2025-04-10T13:03:47.386Z] broad list of issues found
[2025-04-10T13:03:47.485Z] score: 5
⚠️ [2025-04-10T13:03:47.485Z] this log will only exist in github checkrun logs
[2025-04-10T13:03:47.485Z] Reason: The constant FIX_BUTTON_HEIGHT is defined but not used in a meaningful way. It should be used to set the height of the button consistently across the application.
⚠️ [2025-04-10T13:03:47.485Z] rule violated
[2025-04-10T13:03:47.485Z] score: 6
⚠️ [2025-04-10T13:03:47.485Z] this log will only exist in github checkrun logs
[2025-04-10T13:03:47.485Z] Reason: The code does not follow the fail-fast principle. It should return early if the conditions are not met, instead of nesting logic inside else branches.
⚠️ [2025-04-10T13:03:47.685Z] rule violated
[2025-04-10T13:03:47.685Z] score: 4
⚠️ [2025-04-10T13:03:47.685Z] this log will only exist in github checkrun logs
[2025-04-10T13:03:47.685Z] Reason: The string 'No fix available' should be reused from existing strings if possible to maintain consistency and avoid duplication.
[2025-04-10T13:03:47.685Z] score: 7
⚠️ [2025-04-10T13:03:47.685Z] this log will only exist in github checkrun logs
[2025-04-10T13:03:47.685Z] Reason: The use of Platform.runLater should be minimized to avoid potential threading issues. Consider refactoring to ensure UI updates are handled appropriately.
[2025-04-10T13:03:47.685Z] score: 3
⚠️ [2025-04-10T13:03:47.685Z] this log will only exist in github checkrun logs
[2025-04-10T13:03:47.685Z] Reason: The use of AtomicBoolean here is unnecessary as there is no concurrent modification. A simple boolean variable would suffice.
⚠️ [2025-04-10T13:03:47.685Z] rule violated
[2025-04-10T13:03:47.685Z] score: 4
⚠️ [2025-04-10T13:03:47.685Z] this log will only exist in github checkrun logs
[2025-04-10T13:03:47.685Z] Reason: The notification strings should be reused from existing strings to maintain consistency and avoid duplication.
[2025-04-10T13:03:47.685Z] filtering out low importance issues
[2025-04-10T13:03:47.685Z] writing comments to pr...
[2025-04-10T13:03:47.986Z] Skipping comment creation - existing comment found at line 116 in src/main/java/org/jabref/gui/integrity/IntegrityCheckDialog.java
[2025-04-10T13:03:47.986Z] file: src/main/java/org/jabref/gui/integrity/IntegrityCheckDialogViewModel.java
[2025-04-10T13:03:47.986Z] reading file blob
[2025-04-10T13:03:56.787Z] filteredRules: 1. If a method has JavaDoc and code of the method has changed, the JavaDoc has to be updated accordingly. No need to add JavaDoc for "trivial" exceptions
2. If code in org.jabref.model or org.jabref.logic has been changed, tests need to be adapted or updated accordingly
3. The code should follow the fail fast principle by immediately handling invalid states and returning early instead of nesting logic inside else branches.

Example:

Bad:

if (path.isEmpty()) {
return false;
} else {
// other code
}

Good:

if (path.isEmpty()) {
return false;
}

// other code
4. The pull request title should contain a short title of the issue fixed (or what the PR adresses) and not just "Fix issue xyz"
5. The "Mandatory checks" are Markdown TODOs. They should be formatted as that. Wrong: - [ x]. Either - [ ] or - [x].
6. New methods (and new classes) should follow the Single-responsibility principle (SRP).
7. There should be JavaDoc for complex methods.
8. "Magic" numbers or strings should be constants - or at least have a Java comment. Exception: JavaFX height and widths.
9. Avoid code duplication
10. Use modern Java best practices, such as Arguments.of() instead of new Object[] especially in JUnit tests or Path.of() instead of Paths.get(), to improve readability and maintainability.
11. Exceptions should be used for exceptional states - not for normal control flow
12. Follow the principles of "Effective Java"
13. In JabRef, localized strings are done using Localization.lang("string"). More information at https://devdocs.jabref.org/code-howtos/localization.html.
14. No use of Java SWING, only JavaFX is allowed as UI technology
15. @DisplayName for tests should only be used if absolutely necessary: The method name itself should be comprehensive enough.
16. Comments should add new information (e.g. reasoning of the code). It should not be plainly derived from the code itself.

Example for trivail comments:

    // Commit the staged changes
    RevCommit commit = git.commit()

    fieldName = fieldName.trim().toLowerCase(); // Trim and convert to lower case

17. Instead of Files.createTempDirectory @TempDir JUnit5 annotation should be used.
18. Do not catch the general java java.lang.Exception. Catch specific exeptions only
19. Avoid exclamation marks at the end of a sentence. They are more for screaming. Use a dot to end the sentence.
20. All labels and texts should be sentence case (and not title case)
21. New public methods should not return null. They should make use of java.util.Optional. In case null really needs to be used, the JSpecify annotations must be used.
22. Use "BibTeX" as spelling for bibtex in Java strings. In variable names "Bibtex" should be used.
23. New strings should be consistent to other strings. They should also be grouped semantically together.
24. Existings strings should be reused instead of introducing slightly different strings
25. Comments on/above methods should be JavaDoc.not simple Java comments //. Three /// are OK (because this is Java23 and later)
26. The CHANGELOG.md entry should be for end users (and not programmers).
27. User dialogs should have proper button labels: NOT yes/no/cancel, but indicating the action which happens when pressing the button
28. GUI code should only be a gateway to code in org.jabref.logic. More complex code regarding non-GUI operations should go into org.jabref.logic. Think of layerd archicture.
29. null should never be passed to a method (except it has the same name).
30. Do not add extra blank lines in CHANGELOG.md
31. Remove commented code. (To keep a history of changes git was made for.)
32. Do not use Objects.requireNonNull. Use JSpecify @NonNull annotation if needs be.
33. Do not throw unchecked exceptions (e.g., do not throw new RuntimeException, do not throw new IllegalStateException)

Reason: This tears down the whole application. One does not want to loose data only because "a corner" of the application broke.
34. When adding JavaDoc, the text should be non-trivial.

Example for trivial JavaDoc:

 * @param backupDir the backup directory
 * @param dbfile the database file
 * @throws IOException     if an I/O error occurs
 * @throws GitAPIException if a Git API error occurs

35. Code should not be reformatted only because of syntax. There need to be new statements added if reformatting.
36. If @TempDir is used, there is no need to clean it up

Example for wrong code:

    @AfterEach
    void tearDown() throws IOException {
        FileUtils.cleanDirectory(tempDir.toFile());
    }

37. Assert the contents of objects (assertEquals), not checking for some Boolean conditions (assertTrue/assertFalse)

Example for wrong code:

        assertTrue(
                entry.getFiles().stream()
                     .anyMatch(file -> file.getLink().equals(newFile.getFileName().toString()) ||
                             file.getLink().endsWith("/" + newFile.getFileName().toString()))
        );

38. No "new Thread()", use "org.jabref.logic.util.BackgroundTask" and its "executeWith"
39. try blocks shoud cover as less statements as possible (and not whole methods)
40. use "throws Exception" in the method declaration instead of try-catch and fail/log/...
41. When creating a new BibEntry object "withers" should be used: Instead of setField, withField methods should be used.
42. In case Java comments are added, they should match the code following. They should be a high-level summary or guidance of the following code (and not some reandom text or just re-stating the obvious)
43. Use the methods of java.util.Optional. ifPresent.

NOT

Optional<String> resolved = bibEntry.getResolvedFieldOrAlias(...);
String value = resolved.orElse("");
doSomething(value)

Following is fine:

Optional<String> resolved = bibEntry.getResolvedFieldOrAlias(...);
resolved.ifPresent(value -> doSomething(value));

44. If the java.util.Optional is really present, use use get() (and not orElse(""))
45. Use Java Text blocks (""") for multiline string constants
46. Use compiled patterns (Pattern.compile)

Examples:

NOT: x.matches(".\s{2,}.")

BUT:
private final static PATTERN = ...
and then PATTERN.matcher(x)
47. Use placeholders if variance is in localizaiton:

BAD: Localization.lang("Current JabRef version") + ": " + buildInfo.version);

GOOD: Localization.lang("Current JabRef version: %0", buildInfo.version);
48. Log exceptions using exception logging capabilities

BAD: LOGGER.info("Failed to push: ".concat(e.toString()));

GOOD: LOGGER.info("Failed to push", e);
49. Boolean method parameters (for public methods) should be avoided. Better create two distinct methods (which maybe call some private methods)
50. Use modern Java data structures

BAD: new HashSet<>(Arrays.asList(...))

GOOD: Sef.of(...)
51. Use logger conversion to log objects - do not do manual conversion. Especially, java.util.Optional can be logged directly, no need for get(), ,getOrElse(), ...

BAD:

        LOGGER.info(String.valueOf(path.get()));
        LOGGER.info("Not a git repository");

GOOD:

       LOGGER.info("Not a git repository", path);

52. Java 21 introduced SequencedCollection and SequencedSet interfaces. Use it instead of LinkedHashSet (where applicable)
53. Use StringJoiner instead of StringBuilder (if possible)
54. In case defaults are modified, there needs to be a preference migration.
55. Proper localization. In JabRef, localized strings are prefix with %

BAD: <Label ... text="Git"/>

GOOD: <Label ... text="%Git"/>
56. Labels should not end with ":"

BAD:

GOOD:
57. Plain JUnit assert should be used instead of org.assertj (if possible)

BAD: assertThat(gitPreferences.getAutoPushEnabled()).isFalse();

GOOD: assertFalse(gitPreferences.getAutoPushEnabled());
58. Do not catch exceptions in Test - let JUnit handle

BAD: try {...code...} catch (IOException e) {
throw new AssertionError("Failed to set up test directory", e);
}

GOOD: ...code...
59. Name variables using camelcase (and not snake_case, etc)
60. Minimal quality for variable names: Not extraEntry2, extraEntry3; but include meaning/intention into the variable names
61. One should use jabref's dialogService (instead of Java native FileChooser)

dialogService.showFileOpenDialog(fileDialogConfiguration).ifPresent(path -> ...)

and with FileDialogConfiguration offers the Builder pattern.
(see e.g NewLibraryFromPdfAction)

[2025-04-10T13:04:18.286Z] filtering out non-relevant issues
[2025-04-10T13:04:18.286Z] broad list of issues found
⚠️ [2025-04-10T13:04:18.286Z] rule violated
[2025-04-10T13:04:18.287Z] score: 7
⚠️ [2025-04-10T13:04:18.287Z] this log will only exist in github checkrun logs
[2025-04-10T13:04:18.287Z] Reason: The method 'resolveIssue' is public and should not return null. It should use java.util.Optional to handle potential null values.
[2025-04-10T13:04:18.287Z] score: 3
⚠️ [2025-04-10T13:04:18.287Z] this log will only exist in github checkrun logs
[2025-04-10T13:04:18.287Z] Reason: The import statement for Collectors is duplicated in the patch, which is unnecessary and should be removed to avoid redundancy.
⚠️ [2025-04-10T13:04:18.287Z] rule violated
[2025-04-10T13:04:18.287Z] score: 6
⚠️ [2025-04-10T13:04:18.287Z] this log will only exist in github checkrun logs
[2025-04-10T13:04:18.287Z] Reason: The method 'resolveIssue' lacks implementation details and JavaDoc for complex methods should be provided to explain its purpose and usage.
[2025-04-10T13:04:18.287Z] filtering out low importance issues
[2025-04-10T13:04:18.287Z] writing comments to pr...
[2025-04-10T13:04:25.585Z] rule: All labels and texts should be sentence case (and not title case)
⚠️ [2025-04-10T13:04:25.585Z] this log will only exist in github checkrun logs
[2025-04-10T13:04:25.586Z] comment: The method 'resolveIssue' is public and should not return null. It should use java.util.Optional to ...
[2025-04-10T13:04:25.586Z] file: src/main/java/org/jabref/logic/integrity/ASCIICharacterChecker.java
[2025-04-10T13:04:25.586Z] file is outside of new changes on pr, skipping
[2025-04-10T13:04:25.586Z] file: src/main/java/org/jabref/logic/integrity/AbbreviationChecker.java
[2025-04-10T13:04:25.586Z] file is outside of new changes on pr, skipping
[2025-04-10T13:04:25.586Z] file: src/main/java/org/jabref/logic/integrity/BibStringChecker.java
[2025-04-10T13:04:25.616Z] file is outside of new changes on pr, skipping
[2025-04-10T13:04:25.616Z] file: src/main/java/org/jabref/logic/integrity/BibTeXEntryTypeChecker.java
[2025-04-10T13:04:25.616Z] file is outside of new changes on pr, skipping
[2025-04-10T13:04:25.616Z] file: src/main/java/org/jabref/logic/integrity/BooktitleChecker.java
[2025-04-10T13:04:25.616Z] file is outside of new changes on pr, skipping
[2025-04-10T13:04:25.616Z] file: src/main/java/org/jabref/logic/integrity/BracketChecker.java
[2025-04-10T13:04:25.616Z] file is outside of new changes on pr, skipping
[2025-04-10T13:04:25.616Z] file: src/main/java/org/jabref/logic/integrity/CitationKeyDeviationChecker.java
[2025-04-10T13:04:25.616Z] file is outside of new changes on pr, skipping
[2025-04-10T13:04:25.616Z] file: src/main/java/org/jabref/logic/integrity/CitationKeyDuplicationChecker.java
[2025-04-10T13:04:25.616Z] file is outside of new changes on pr, skipping
[2025-04-10T13:04:25.616Z] file: src/main/java/org/jabref/logic/integrity/DateChecker.java
[2025-04-10T13:04:25.616Z] file is outside of new changes on pr, skipping
[2025-04-10T13:04:25.616Z] file: src/main/java/org/jabref/logic/integrity/DoiDuplicationChecker.java
[2025-04-10T13:04:25.616Z] file is outside of new changes on pr, skipping
[2025-04-10T13:04:25.616Z] file: src/main/java/org/jabref/logic/integrity/DoiValidityChecker.java
[2025-04-10T13:04:25.616Z] file is outside of new changes on pr, skipping
[2025-04-10T13:04:25.616Z] file: src/main/java/org/jabref/logic/integrity/EditionChecker.java
[2025-04-10T13:04:25.616Z] file is outside of new changes on pr, skipping
[2025-04-10T13:04:25.616Z] file: src/main/java/org/jabref/logic/integrity/EntryLinkChecker.java
[2025-04-10T13:04:25.616Z] file is outside of new changes on pr, skipping
[2025-04-10T13:04:25.616Z] file: src/main/java/org/jabref/logic/integrity/FileChecker.java
[2025-04-10T13:04:25.616Z] file is outside of new changes on pr, skipping
[2025-04-10T13:04:25.616Z] file: src/main/java/org/jabref/logic/integrity/HTMLCharacterChecker.java
[2025-04-10T13:04:25.616Z] file is outside of new changes on pr, skipping
[2025-04-10T13:04:25.616Z] file: src/main/java/org/jabref/logic/integrity/HowPublishedChecker.java
[2025-04-10T13:04:25.616Z] file is outside of new changes on pr, skipping
[2025-04-10T13:04:25.617Z] file: src/main/java/org/jabref/logic/integrity/ISBNChecker.java
[2025-04-10T13:04:25.617Z] file is outside of new changes on pr, skipping
[2025-04-10T13:04:25.617Z] file: src/main/java/org/jabref/logic/integrity/ISSNChecker.java
[2025-04-10T13:04:25.617Z] file is outside of new changes on pr, skipping
[2025-04-10T13:04:25.617Z] file: src/main/java/org/jabref/logic/integrity/IntegrityIssue.java
[2025-04-10T13:04:25.617Z] file is outside of new changes on pr, skipping
[2025-04-10T13:04:25.617Z] file: src/main/java/org/jabref/logic/integrity/IntegrityMessage.java
[2025-04-10T13:04:25.617Z] file is outside of new changes on pr, skipping
[2025-04-10T13:04:25.617Z] file: src/main/java/org/jabref/logic/integrity/JournalInAbbreviationListChecker.java
[2025-04-10T13:04:25.617Z] file is outside of new changes on pr, skipping
[2025-04-10T13:04:25.617Z] file: src/main/java/org/jabref/logic/integrity/MonthChecker.java
[2025-04-10T13:04:25.617Z] file is outside of new changes on pr, skipping
[2025-04-10T13:04:25.617Z] file: src/main/java/org/jabref/logic/integrity/NoBibtexFieldChecker.java
[2025-04-10T13:04:25.617Z] file is outside of new changes on pr, skipping
[2025-04-10T13:04:25.617Z] file: src/main/java/org/jabref/logic/integrity/NoteChecker.java
[2025-04-10T13:04:25.617Z] file is outside of new changes on pr, skipping
[2025-04-10T13:04:25.617Z] file: src/main/java/org/jabref/logic/integrity/PagesChecker.java
[2025-04-10T13:04:25.617Z] file is outside of new changes on pr, skipping
[2025-04-10T13:04:25.617Z] files scanned: 2
[2025-04-10T13:04:25.617Z] lines scanned: 268
[2025-04-10T13:04:25.617Z] rules for project: 61
[2025-04-10T13:04:25.617Z] issues caught by your rules: 1