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

Build modernisation to modern Gradle/Android Studio #86

Merged
merged 8 commits into from
Oct 26, 2023

Conversation

TheRealAgentK
Copy link
Contributor

Summary

What problem(s) does this PR solve?

Fixes the AS Gradle builds in so far that it's now up to date again with current Android studio version.

Type of change

  • Internal to project

What was done to solve the problem?

  • Updated Gradle and AGP versions progressively and fixed issues arising along the way
  • Replaced KAE (deprecated now) with view binding in sample app
  • Set version to 4.1-SNAPSHOT for the time being (more changes to come)

QA

  • Project should be able to import in current stable AS again and sync/build
  • :app and :provider modules (the latter via the app build) build correctly when running the sample app in an Emulator (tested with SDK 33 for now)

// apply from: 'gradle-mvn-push.gradle'
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can we remove this all together or write a code comment as to why it's commented out

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, I just commented it out because this contains all the gradle tasks for publishin the library builds to repo servers and that flow doesn't work at the moment. But it will be brought back.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Ideally would be good to add that as a code comment but if it's in near future (upcoming PR?) then no worries :)

Copy link
Collaborator

@sumitramanga sumitramanga left a comment

Choose a reason for hiding this comment

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

Looks good so far! Just some comments to mainly clear up my confusion before I give the approval on this one

Copy link
Collaborator

@sumitramanga sumitramanga left a comment

Choose a reason for hiding this comment

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

LGTM :shipit:

@TheRealAgentK TheRealAgentK merged commit ef90d94 into master Oct 26, 2023
@TheRealAgentK TheRealAgentK deleted the feature/buildModernisation branch October 26, 2023 19:23
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