Skip to content

Conversation

@mdekrey
Copy link
Contributor

@mdekrey mdekrey commented Dec 26, 2018

Updates gradle to use React-Native and Android's latest standards for improving dependency management.

Resolves #251:

  • By changing compile to implementation, we comply with Android's new dependency configurations. implementation is slightly faster than api, and the purpose of this library is to expose admob, not its underlying dependencies (like Google Play), to anyone who adds it to their project.
  • By using default values for target SDK, etc., we fix dependency resolution issues that may occur when combining with other native libs. The new gradle template generated by the various CLI tools include an ext block to add these default properties, but if anyone is using older versions and does not have that block, this behavior will not change for them. (In case anyone wants to manually add this to an existing project, this is the best doc I've found.)

Updates gradle to use React-Native and Android's latest standards for improving dependency management.
@mdekrey mdekrey mentioned this pull request Dec 27, 2018
@jugutier
Copy link

Love the safeExtGet. Thanks!

@koenpunt
Copy link
Collaborator

koenpunt commented Sep 15, 2019

Thanks! And sorry for the wait.

roy8119425 pushed a commit to roy8119425/react-native-admob that referenced this pull request Apr 14, 2021
Updates gradle to use React-Native and Android's latest standards for improving dependency management.
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.

Changes needed for gradle 3.0.1 compatibility

3 participants