Skip to content

Conversation

@svlilyas
Copy link

  • Gradle version upgraded to 7.4.2 and Android Gradle Plugin upgraded to 7.2.2
  • TargetSdk and CompileSdk upgraded to 32
  • buildSrc added to manage Plugins and common Android Configs
  • VersionCatalogs implemented with TOML file
  • TOML file updated

Gradle upgraded to version 7.4.2 and Android Gradle Plugin Upgraded to version 7.2.2
test-app version, targetSdk, compileSdk upgraded
streamer library dependencies upgrade
opds library dependencies upgraded
navigator-media2 library dependencies upgraded.
libraries upgraded
…ed TOML Version Catalogs.

Added buildSrc to manage Android Configs from one place and  implemented TOML  Version Catalogs.
TOML library implementation
AndroidConfig variables used in gradles
Copy link
Member

@mickael-menu mickael-menu left a comment

Choose a reason for hiding this comment

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

@svlilyas Thank you for taking the time to submit a PR, there's good stuff in it. However, please open an issue (or a Slack message) first next time to discuss the changes, to make sure you don't waste your time on work that won't be accepted.

I left a few comments, but could you also rebase your changes to merge the PR onto the develop branch? We only merge hotfixes on main, any other changes must be on develop.

@svlilyas
Copy link
Author

svlilyas commented Sep 9, 2022

I could not able to find rebase settings. Can you tell me how to do it?

@mickael-menu mickael-menu changed the base branch from main to develop September 9, 2022 14:53
@mickael-menu
Copy link
Member

I changed the base branch of your PR to develop, try running git merge develop from your clone and fix any conflict you will probably get. This or alternatively you can start again from develop with a new PR and copy your changes there.

@svlilyas svlilyas closed this Sep 18, 2022
@mickael-menu
Copy link
Member

@svlilyas Looks like Android Studio Dolphin has better support for TOML version catalogs 👍

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