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

Revert "fix(deps): update dependency org.sonarsource.java:sonar-java-… #828

Closed
wants to merge 1 commit into from

Conversation

hazendaz
Copy link
Member

…plugin to v7"

This reverts commit 397b4e2.

@hazendaz hazendaz self-assigned this Oct 31, 2023
@hazendaz hazendaz closed this Oct 31, 2023
@gtoison
Copy link
Contributor

gtoison commented Oct 31, 2023

Thanks!

@hazendaz
Copy link
Member Author

think that was the wrong one, fixing the right one. I think this one is ok. Can you confirm?

@hazendaz
Copy link
Member Author

opened #829 instead.

@gtoison
Copy link
Contributor

gtoison commented Oct 31, 2023

The project uses two sonar API: the plugin API and the Java plugin API
Both should remain on the version corresponding to the earliest SonarQube version supported.
For the Java plugin API that's 5.13.1.18282 and for the plugin API that's 7.9.6

Note that at this point SonarQube 7.9 and 8.9 aren't supported by Sonarsource anymore, however the plugin still works.
Someday they will make a breaking change and we'll need to drop support for these older versions

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