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

fix androidX legacy dependencies... #246

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

diegojimenes
Copy link

this is a hacky way to detect when to use the legacy dependecies
from androidX.

let me know if there is a better way.

this is a hacky way to detect when to use the legacy dependecies
from androidX.
@diegojimenes diegojimenes force-pushed the fix-androidx-legacy-dependecies branch from 9682253 to d6c3a4b Compare July 18, 2019 19:07
@diegojimenes
Copy link
Author

also, people might need to run this command npx jetify && npx react-native run-android.

@congnguyen91
Copy link
Contributor

I think this PR should be checked :) @tolu360 now React Native 0.60 has released & supported Android-X. A few of modules upgraded. There are two options:

  1. Only support android-x for the next version (if the current version was stable).
  2. Support both of non-android-x & android-x config.

@diegojimenes
Copy link
Author

Yes, @congnguyen91. This should keep compatibility with both, for now.
Maybe later, we can drop the support for non-androidX.

@tolu360
Copy link
Owner

tolu360 commented Jul 19, 2019

Hi guys! I would take a look at this PR, thanks @diegojimenes! I do prefer we keep support for both for the nearest future - I may release androidX support on a beta for now. I haven't even played with 0.60.* myself, but would surely make out time.

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.

3 participants