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

Sonar 10 #2612

Closed
wants to merge 4 commits into from
Closed

Sonar 10 #2612

wants to merge 4 commits into from

Conversation

agebert
Copy link
Contributor

@agebert agebert commented Jan 18, 2024

With these changes you can use 10.x version of org.sonarsource.sonarqube:sonar-plugin-api-impl and
org.sonarsource.api.plugin:sonar-plugin-api .

I fixed all compile issues with the help of https://sonarsource.atlassian.net/browse/SONARTEXT-57 .

I did not test the result.


This change is Reviewable

….15.0.435

Refactor the code to use new artifact sonar-plugin-api-test-fixtures.
Get rid of deprecated LoggerLevel.
…0.3.0.1951

And bump also org.sonarsource.api.plugin:sonar-plugin-api-test-fixtures

Some classes move to org.sonar.api.testfixtures.measure package.
… to 10.3.0.82913

And bump org.sonarsource.sonarqube:sonar-testing-harness
@guwirth guwirth self-requested a review January 18, 2024 21:38
@guwirth
Copy link
Collaborator

guwirth commented Jan 18, 2024

Hi @agebert, thx for providing this. I started the CI with it …

@guwirth
Copy link
Collaborator

guwirth commented Jan 18, 2024

Hi @agebert,

the PR is running with our current CI. The current settings does not test with Java 17 and SQ 10.

The file have to be changed is:
https://github.com/SonarOpenCommunity/sonar-cxx/blob/master/.github/workflows/cxx-ci.yml

Regards

@guwirth
Copy link
Collaborator

guwirth commented Jan 19, 2024

@agebert I will provide an adapted CI file in the course of the day …

@guwirth
Copy link
Collaborator

guwirth commented Jan 19, 2024

Hello @agebert,

I added PR #2613 with modified CI files. You should use this files and merge it into this PR #2612 to add testing with SQ 10. The tests are for SQ 9.9 LTS and SQ 10 (SQ 8.9 LTS removed), not sure if your changes are working with boths? If not you can remove the SQ 9.9 tests.

Regards,

@agebert agebert mentioned this pull request Jan 22, 2024
@guwirth guwirth closed this Jan 24, 2024
@agebert agebert deleted the sonar-10 branch February 2, 2024 22:57
@guwirth guwirth added this to the 2.2.0 milestone Mar 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants