Skip to content
Merged
Show file tree
Hide file tree
Changes from 9 commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 11 additions & 0 deletions .github/workflows/tests-code.yml
Original file line number Diff line number Diff line change
Expand Up @@ -114,6 +114,17 @@ jobs:
run: |
./gradlew --no-configuration-cache :rewriteDryRun

format:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v5
- uses: koppor/intellij-idea-format@update-to-latest-intellij
with:
file-mask: "*.java"
settings-file: "config/IntelliJ Code Style.xml"
# job should fail in case there are changes
- run: git diff-files --quiet

modernizer:
name: Modernizer
runs-on: ubuntu-latest
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -208,9 +208,7 @@ public String summarizeOneDocument(String filePath, String document) throws Inte

int passes = 0;

// @formatter:off
do {
// @formatter:on
do{
passes++;
LOGGER.debug("Summarizing chunk(s) for file \"{}\" of entry {} ({} pass)", filePath, citationKey, passes);

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,10 +19,10 @@
public class LibrariesResource {

@Inject
private SrvStateManager srvStateManager;
private SrvStateManager srvStateManager;
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The patch reformats code without adding new statements, which contradicts the guideline that code should not be reformatted only because of syntax. This change should be reverted unless new statements are added.


@Inject
private FilesToServe filesToServe;
private FilesToServe filesToServe;
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The patch reformats code without adding new statements, which contradicts the guideline that code should not be reformatted only because of syntax. This change should be reverted unless new statements are added.


@Inject
private Gson gson;
Expand Down
Loading