Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add qodana CI checks #1

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

qodana-cloud[bot]
Copy link

@qodana-cloud qodana-cloud bot commented Sep 26, 2024

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

Copy link

Qodana Community for JVM

271 new problems were found

Inspection name Severity Problems
Unused symbol 🔶 Warning 90
Unresolved reference in KDoc 🔶 Warning 7
Unstable API Usage 🔶 Warning 6
Usage of redundant or deprecated syntax or deprecated symbols 🔶 Warning 4
Single character alternation 🔶 Warning 3
'Enum.values()' is recommended to be replaced by 'Enum.entries' since 1.9 🔶 Warning 2
Leaking 'this' in constructor 🔶 Warning 1
Recursive property accessor 🔶 Warning 1
Unused receiver parameter 🔶 Warning 1
Class member can have 'private' visibility ◽️ Notice 93
Function naming convention ◽️ Notice 12
Redundant 'if' statement ◽️ Notice 8
Argument could be converted to 'Set' to improve performance ◽️ Notice 6
Convert 'object' to 'data object' ◽️ Notice 6
Local variable naming convention ◽️ Notice 6
Unnecessary type argument ◽️ Notice 6
Constant conditions ◽️ Notice 4
Property naming convention ◽️ Notice 3
Function or property has platform type ◽️ Notice 2
Non-distinguishable logging calls ◽️ Notice 2
Nested lambda has shadowed implicit parameter ◽️ Notice 2
Const property naming convention ◽️ Notice 1
Enum entry naming convention ◽️ Notice 1
Join declaration and assignment ◽️ Notice 1
Redundant overriding method ◽️ Notice 1
Redundant 'constructor' keyword ◽️ Notice 1
Negated call can be simplified ◽️ Notice 1

☁️ View the detailed Qodana report

Contact Qodana team

Contact us at [email protected]

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