Skip to content

Test craft#56

Merged
jaksonlin merged 5 commits intomainfrom
test-craft
May 7, 2025
Merged

Test craft#56
jaksonlin merged 5 commits intomainfrom
test-craft

Conversation

@jaksonlin
Copy link
Copy Markdown
Owner

fix annotation generation hang

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 6, 2025

Qodana Community for JVM

93 new problems were found

Inspection name Severity Problems
Unused import 🔶 Warning 11
Unnecessary call to 'toString()' 🔶 Warning 10
Constant values 🔶 Warning 8
Nullability and data flow problems 🔶 Warning 7
Field may be 'final' 🔶 Warning 7
Unchecked warning 🔶 Warning 6
Field can be local 🔶 Warning 5
Result of method call ignored 🔶 Warning 5
@NotNull/@Nullable problems 🔶 Warning 5
Call to 'printStackTrace()' 🔶 Warning 4
Minimum 'switch' branches 🔶 Warning 3
Incorrect string capitalization 🔶 Warning 2
Redundant type cast 🔶 Warning 2
Redundant 'if' statement 🔶 Warning 2
'InputStream' and 'OutputStream' can be constructed using 'Files' methods 🔶 Warning 1
Mismatched query and update of collection 🔶 Warning 1
Redundant 'Collection' operation 🔶 Warning 1
Stream API call chain can be simplified 🔶 Warning 1
Redundant local variable 🔶 Warning 1
Unnecessary 'return' statement 🔶 Warning 1
Unnecessary semicolon 🔶 Warning 1
Unused assignment 🔶 Warning 1
Write-only object 🔶 Warning 1
Non-distinguishable logging calls ◽️ Notice 3
Non-constant string concatenation as argument to logging call ◽️ Notice 3
Method can be extracted ◽️ Notice 1

💡 Qodana analysis was run in the pull request mode: only the changed files were checked

View the detailed Qodana report

To be able to view the detailed Qodana report, you can either:

To get *.log files or any other Qodana artifacts, run the action with upload-result option set to true,
so that the action will upload the files as the job artifacts:

      - name: 'Qodana Scan'
        uses: JetBrains/qodana-action@v2024.1.5
        with:
          upload-result: true
Contact Qodana team

Contact us at qodana-support@jetbrains.com

@jaksonlin
Copy link
Copy Markdown
Owner Author

#55 is also fixed in this PR, using a dedicated thread pool to run the git command

@jaksonlin jaksonlin merged commit fa43765 into main May 7, 2025
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant