-
Notifications
You must be signed in to change notification settings - Fork 4
ci(CA): Add qodana static analysis #76
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
Conversation
Qodana Community for JVM490 new problems were found
☁️ View the detailed Qodana report Contact Qodana teamContact 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 |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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
Adds qodana.yaml and GitHub workflow file that runs Qodana static analysis on each pull request.