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

Migrate Android Support Libraries To AndroidX #589

Closed
wants to merge 7 commits into from

Conversation

aarongrider
Copy link

@aarongrider aarongrider commented Apr 27, 2019

This is a new PR following: #579

Currently I am using 2 forked dependencies to get the Android tests to pass. We will probably want to wait till our dependencies take the AndroidX upgrade and install directly from npm before merging this branch.

callstack/react-native-slider#56
callstack/react-native-pager-view#22

@osdnk
Copy link
Contributor

osdnk commented Apr 27, 2019

Supercool. Is there anything we should be afraid of merging it? Are we going drop support for some versions with it?

@aarongrider
Copy link
Author

As far as I understand AndroidX libraries are not compatible with non AndroidX projects, so all projects that consume this library must also be migrated to AndroidX.

I wanted to leave it to the maintainers to decide if this is something they would want to do. Unfortunately, the reverse is also true, any Android projects that have migrated to AndroidX, cannot use this library without these changes.

Do we have any idea how other RN libraries are handling this situation?

@aarongrider
Copy link
Author

It looks like Flutter has depreciated support library support in favor of AndroidX and has included instructions on how to upgrade your project, along with a list of the last plugin versions to support the legacy support libraries: https://flutter.dev/docs/development/packages-and-plugins/androidx-compatibility

This might be a good approach?

@aarongrider
Copy link
Author

There is also a React Native AndroidX migration plan here:
facebook/react-native#23112

@osdnk
Copy link
Contributor

osdnk commented Apr 29, 2019

@aarongrider Please apply changes in your PR to Slider

@aarongrider
Copy link
Author

Should we just wait for RN 60?

Co-Authored-By: Leon Omelan <rozpierog@gmail.com>
@kmagiera
Copy link
Member

Hey, I'd love to finally get that androidX migration in, however this PR has too many conflicts I am not sure how to resolve. Do you mind updating it?

@se-bastiaan
Copy link

se-bastiaan commented Aug 3, 2019

Of all the AndroidX PRs this seems to be the most complete one. It updates the example app and all dependencies and keeps the CI intact.

Instead of using forked versions of the viewpager and slider for the example app I would suggest using https://github.com/mikehardy/jetifier instead though and update the react-native version to 0.60+.

It would probably also be good to make it possible for users of the package to provide a version of the AndroidX packages instead of hardcoding the version like was done in react-native-device-info. (ignore the backwards compatibility part though, you don't want that here)

@osdnk
Copy link
Contributor

osdnk commented Oct 31, 2019

we already merged another PR switching to AndroidX

@osdnk osdnk closed this Oct 31, 2019
@aarongrider aarongrider deleted the androidx branch November 1, 2019 00:03
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.

None yet

5 participants