Skip to content

Conversation

@dulmandakh
Copy link
Contributor

@dulmandakh dulmandakh commented Aug 21, 2019

This PR makes Gesture Handler compatible with AndroidX, by using Nullable annotation and Pools from AndroidX. But also removes buildToolsVersion as it's no longer required by Android Gradle Plugin since version 3.1.0, see https://developer.android.com/studio/releases/gradle-plugin#3-1-0, and bumped compileSdkVersion, targetSdkVersion to 28.

Upgraded Example app, and had to bump Android Gradle Version, thus Gradle wrapper version. Also had to run jetify because some dependencies are not AndroidX ready.

Copy link
Member

@kmagiera kmagiera left a comment

Choose a reason for hiding this comment

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

Thanks @dulmandakh

@dulmandakh
Copy link
Contributor Author

Android CI is green 👍

@dulmandakh
Copy link
Contributor Author

@kmagiera CI is green, and good to be merged 😎

@kmagiera kmagiera merged commit ed350dc into software-mansion:master Aug 22, 2019
@kmagiera
Copy link
Member

Awesome job @dulmandakh – thanks so much for your help!

janicduplessis pushed a commit to janicduplessis/react-native-gesture-handler that referenced this pull request Feb 16, 2020
This PR makes Gesture Handler compatible with AndroidX, by using Nullable annotation and Pools from AndroidX. But also removes **buildToolsVersion** as it's no longer required by Android Gradle Plugin since version 3.1.0, see https://developer.android.com/studio/releases/gradle-plugin#3-1-0, and bumped compileSdkVersion, targetSdkVersion to 28.

Upgraded **Example** app, and had to bump Android Gradle Version, thus Gradle wrapper version. Also had to run **jetify** because some dependencies are not AndroidX ready.
@bananer
Copy link

bananer commented Mar 13, 2020

Hi, I'm wondering why android/lib/build.gradle still points to com.android.support:appcompat-v7 depdency. Should this not have been changed to androidx.appcompat:appcompat? It might be jetifier fixing it for builds automatically...

compile 'com.android.support:appcompat-v7:23.4.0'

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.

4 participants