Skip to content

add Qodana static code analysis #4493

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

arturcic
Copy link
Member

This pull request introduces Qodana code quality analysis to the project. The most important changes include adding a GitHub Actions workflow to run Qodana scans and configuring Qodana analysis through a YAML file.

Qodana Integration:

  • .github/workflows/qodana_code_quality.yml: Added a new GitHub Actions workflow to run Qodana scans on workflow dispatch, pull requests, and pushes to the main branch. The workflow uses the JetBrains/qodana-action to perform the scans.

  • src/qodana.yaml: Configured Qodana analysis with a specified IDE code, inspection profile, and a comprehensive list of included inspections to ensure thorough code quality checks.
    [Copilot is generating a summary...]

@asbjornu
Copy link
Member

The action is failing, but I approve the intent and the code looks fine! 👍🏼

@arturcic
Copy link
Member Author

The action is failing, but I approve the intent and the code looks fine! 👍🏼

I want to spend a bit of time on this and because we already have Jetbrains in the team, let's use it as well. Keeping it as draft for now

@arturcic arturcic force-pushed the main branch 8 times, most recently from 2a50302 to 18ca297 Compare April 16, 2025 14:33
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.

2 participants