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

SCANGRADLE-178 increase android sdk to be compatible with android gradle 8.3.0 #258

Closed
wants to merge 2 commits into from

Conversation

erwan-serandour
Copy link
Contributor

@erwan-serandour erwan-serandour commented Nov 5, 2024

SCANGRADLE-178

The SonarSource Team

@hashicorp-vault-sonar-prod hashicorp-vault-sonar-prod bot changed the title increase android sdk to be compatible with android gradle 8.3.0 SCANGRADLE-178 increase android sdk to be compatible with android gradle 8.3.0 Nov 5, 2024
Copy link

sonarqube-next bot commented Nov 5, 2024

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but the title of the PR needs to be changed to match the ticket it refers to because as the current proposal does not test with AGP 8.3.0.
I think the PR should be renamed and the commits split in two parts with the first bumping the SDK version of the test projects and the second re-enabling the tests with AGP 7.1.0 and Gradle 7.5.1 (both commits can refer to the same ticket).

Comment on lines +125 to +126
- GRADLE_VERSION: 7.5.1
ANDROID_GRADLE_VERSION: 7.1.0

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Consider re-enabling that configuration as part of a separate commit

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it is a good idea, I thought about splitting but was a bit too lazy. 👍

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