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

Upgrade to Gradle 8.6 #104

Merged
merged 4 commits into from
Apr 8, 2024
Merged

Upgrade to Gradle 8.6 #104

merged 4 commits into from
Apr 8, 2024

Conversation

cdr-chakotay
Copy link
Contributor

@cdr-chakotay cdr-chakotay commented Mar 27, 2024

1.)First Upgrade to Gradle 8.4

  • Upgrade Gradle itself
  • Update Manifest files and use namespaces instead. => Now package info is injected dynamically
  • Upgrade Android Gradle plugin to newest version under the following requirements:
    • android.defaults.buildfeatures.buildconfig=false
    • android.enableR8.fullMode=true
    • android.nonTransitiveRClass=true
    • android.nonFinalResIds=true

2.)Then Upgrade to Gradle 8.6
3.) Define a Gradle task to include AAR builds inside the release.
4.) Bumped up Java to Version 17 and 21 for CI Operations (minimum of Java 17 is required by the Android Gralde Plugin)

Added myself to the maintainers of this, we can also skip that if you want :)

- Upgrade Gradle itself
- Update Manifest files and use namespaces instead.
@cdr-chakotay cdr-chakotay self-assigned this Mar 27, 2024
- Upgrade Gradle itself
- Include AAR Release by force
@cdr-chakotay cdr-chakotay requested a review from Acconut March 27, 2024 16:42
@cdr-chakotay cdr-chakotay marked this pull request as ready for review March 27, 2024 16:42
Copy link
Member

@Acconut Acconut left a comment

Choose a reason for hiding this comment

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

This looks good, thanks for working on this!

@cdr-chakotay cdr-chakotay merged commit 8a31da1 into main Apr 8, 2024
3 checks passed
@cdr-chakotay cdr-chakotay deleted the upgrade-gradle branch April 8, 2024 11:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

2 participants