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

Update Qodana configuration after EAP linter's license expiration #172

Merged
merged 8 commits into from
Nov 1, 2023

Conversation

rob93c
Copy link
Member

@rob93c rob93c commented Nov 1, 2023

@rob93c rob93c added the cicd The change affects CI/CD flows label Nov 1, 2023
@rob93c rob93c self-assigned this Nov 1, 2023
@rob93c
Copy link
Member Author

rob93c commented Nov 1, 2023

Although this change will bring back the usual false positives (Patterns in switch are not supported at language level '20'), we need it because currently the action is failing using the EAP and it's preventing changes being deployed.

Once the linter jetbrains/qodana-jvm:2023.3 is released, we should be able to update it and set the languageLevel to JDK_21

@rob93c rob93c marked this pull request as ready for review November 1, 2023 13:01
@rob93c rob93c requested a review from MartelliEnrico November 1, 2023 13:01
@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

@rob93c rob93c merged commit 544717d into main Nov 1, 2023
@rob93c rob93c deleted the update-qodana-linter branch November 1, 2023 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cicd The change affects CI/CD flows
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant