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

Bump AGP to 7.0.2 #1650

Merged
merged 11 commits into from
Sep 14, 2021
Merged

Bump AGP to 7.0.2 #1650

merged 11 commits into from
Sep 14, 2021

Conversation

marandaneto
Copy link
Contributor

@marandaneto marandaneto commented Aug 17, 2021

📜 Description

Bump AGP to 7.0.2

💡 Motivation and Context

Closes #1646

💚 How did you test it?

CI/running

📝 Checklist

  • I reviewed the submitted code
  • I added tests to verify the changes
  • I updated the docs if needed
  • No breaking changes

🔮 Next steps

sentry.properties Outdated Show resolved Hide resolved
@marandaneto
Copy link
Contributor Author

@maciejwalkowiak can you confirm that compiling the Java projects with Java 11 but target/source being Java 8 won't cause any harm? since the new Android Gradle Plugin requires Java 11, either we split up in 2 Gradle projects or we use Java 11 targeting Java 8
Maybe the Matrix instead of Java 8 and 11 now could be Java 11 and 16 or 17

@marandaneto
Copy link
Contributor Author

spotless hits the issue diffplug/spotless#834 if using Java 16

@@ -19,7 +19,7 @@ jobs:
# TODO: windows-latest
os: [ubuntu-latest, macos-latest]
# Zulu Community distribution of OpenJDK
java: [ '8', '11' ]
java: [ '11' ]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

didn't remove the matrix because we want to add Java 16 later

@marandaneto marandaneto changed the title Bump AGP to 7.0.0 Bump AGP to 7.0.2 Sep 14, 2021
@maciejwalkowiak
Copy link
Contributor

"target" means the latest Java version runtime we want to support. I don't see any way it can cause any harm.

@marandaneto marandaneto marked this pull request as ready for review September 14, 2021 11:02
@marandaneto marandaneto merged commit 13ebeb3 into main Sep 14, 2021
@marandaneto marandaneto deleted the bump/agp7.0.0 branch September 14, 2021 12:44
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.

Bump AGP to 7.0.0
3 participants