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

android: allow root project to specify dependency versions #149

Merged
merged 3 commits into from
Apr 25, 2018

Conversation

mlc
Copy link
Contributor

@mlc mlc commented Apr 22, 2018

React Native projects with multiple native dependencies often end up unintentionally relying on a variety of Android SDK versions, which can lead to unpredictable build-time failures or even run-time crashes, such as the weirdness in #67. Many libraries (see, e.g., react-native-maps/react-native-maps#2047, invertase/react-native-firebase#1007) have been moving to a system of allowing the solution recommended by google of having the root project set properties with the desired dependency versions. This PR implements that solution for react-native-auth0.

@demoran23
Copy link

This did the trick for me, after setting supportLibVersion in my root build.gradle's ext block.

lbalmaceda
lbalmaceda previously approved these changes Apr 24, 2018
Copy link
Contributor

@lbalmaceda lbalmaceda left a comment

Choose a reason for hiding this comment

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

It looks fine. Thanks

@lbalmaceda
Copy link
Contributor

after setting supportLibVersion in my root build.gradle's ext block.

@mlc @demoran23 should we add this notice somewhere in the Readme file??

@mlc
Copy link
Contributor Author

mlc commented Apr 25, 2018

Good idea, @lbalmaceda! I've updated the PR to include a note in the README.

Copy link
Contributor

@lbalmaceda lbalmaceda left a comment

Choose a reason for hiding this comment

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

LGTM 🎉

@lbalmaceda lbalmaceda merged commit 31c838d into auth0:master Apr 25, 2018
@lbalmaceda lbalmaceda added this to the v1-Next milestone Apr 25, 2018
@mlc
Copy link
Contributor Author

mlc commented Apr 25, 2018

thanks!

@mlc mlc deleted the gradle_conf_in_root_project branch April 25, 2018 21:43
@Amnesthesia
Copy link

I'm stuck on this but have copied the gradle file from this PR — any update on when this will be published on NPM?

@lbalmaceda lbalmaceda modified the milestones: v1-Next, v1.3.0 Jul 17, 2018
@lbalmaceda
Copy link
Contributor

@Amnesthesia Sorry about that 😞will be in 1.3.0 soon when that release PR get's approved ⚡️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants