Skip to content

Conversation

@qodana-cloud
Copy link

@qodana-cloud qodana-cloud bot commented Jan 26, 2026

Adds qodana.yaml and GitHub workflow file that runs Qodana static analysis on each pull request.

@Nudelsuppe42 Nudelsuppe42 requested a review from Zoriot January 26, 2026 16:42
@Nudelsuppe42 Nudelsuppe42 self-assigned this Jan 26, 2026
@Nudelsuppe42 Nudelsuppe42 added the 🦐 small size This is a small issue. label Jan 26, 2026
@Nudelsuppe42 Nudelsuppe42 changed the title Add qodana CI checks ci(CA): Add qodana static analysis Jan 26, 2026
@github-actions
Copy link

Qodana Community for JVM

490 new problems were found

Inspection name Severity Problems
Usage of API marked for removal 🔴 Failure 8
Deprecated API usage 🔶 Warning 236
Unused import 🔶 Warning 31
Call to 'printStackTrace()' 🔶 Warning 30
Lombok @Getter may be used 🔶 Warning 25
@NotNull/@Nullable problems 🔶 Warning 19
Lombok @Setter may be used 🔶 Warning 14
Nullability and data flow problems 🔶 Warning 13
Field can be local variable 🔶 Warning 13
Constant values 🔶 Warning 9
'size() == 0' can be replaced with 'isEmpty()' 🔶 Warning 9
Unchecked warning 🔶 Warning 8
Javadoc declaration problems 🔶 Warning 7
Field may be 'final' 🔶 Warning 6
'default' not last case in 'switch' 🔶 Warning 5
Statement with empty body 🔶 Warning 5
Result of method call ignored 🔶 Warning 4
Reflective access to non-existent or not visible class member 🔶 Warning 4
Declaration has problems in Javadoc references 🔶 Warning 3
Redundant type cast 🔶 Warning 3
Redundant local variable 🔶 Warning 3
C-style array declaration 🔶 Warning 2
Mismatched query and update of collection 🔶 Warning 2
Stream API call chain can be simplified 🔶 Warning 2
String comparison using '==', instead of 'equals()' 🔶 Warning 2
Concatenation with empty string 🔶 Warning 2
Unnecessary semicolon 🔶 Warning 2
Unnecessary call to 'toString()' 🔶 Warning 2
Unstable API Usage 🔶 Warning 2
Call to 'Arrays.asList()' with too few arguments 🔶 Warning 1
AutoCloseable used without 'try'-with-resources 🔶 Warning 1
Catch block may ignore exception 🔶 Warning 1
Infinite recursion 🔶 Warning 1
Inner class may be 'static' 🔶 Warning 1
Redundant 'close()' 🔶 Warning 1
Static initializer references subclass 🔶 Warning 1
Redundant 'String' operation 🔶 Warning 1
Unused assignment 🔶 Warning 1
Commented out code ◽️ Notice 6
Non-strict inequality '>=' or '<=' can be replaced with '==' ◽️ Notice 3
Method can be extracted ◽️ Notice 1

☁️ View the detailed Qodana report

Contact Qodana team

Contact us at qodana-support@jetbrains.com

ref: ${{ github.event.pull_request.head.sha }} # to check out the actual pull request commit, not the merge commit
fetch-depth: 0 # a full history is required for pull request analysis
- name: 'Qodana Scan'
uses: JetBrains/qodana-action@v2025.2
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please use latest release (v2025.3) https://github.com/JetBrains/qodana-action/releases/tag/v2025.3.1
Or is that not supported?

Copy link
Contributor

Choose a reason for hiding this comment

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

This is a automatic PR, im not gonna interfere with its setup, but check dev manager chat

- name: 'Qodana Scan'
uses: JetBrains/qodana-action@v2025.2
with:
pr-mode: false
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why no pr analysis? If possible, it should also analyze pr's and also that we have a condition that it must pass on new code
Should be possible to set this to true on prs

@Nudelsuppe42 Nudelsuppe42 merged commit 2bafcfd into main Jan 26, 2026
3 checks passed
@Nudelsuppe42 Nudelsuppe42 added this to the Release v0.2.0 milestone Jan 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🦐 small size This is a small issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants