Skip to content

Conversation

lxcid
Copy link

@lxcid lxcid commented Sep 15, 2018

This is similar to #27 but use a cleaner way to achieve the same result.

As of 0.56, we RN proj start declaring a few constants that libraries can query about.

Some library uses this technique so I'm porting it over here:

sandfox pushed a commit to ConnectedHomes/react-native-version-number that referenced this pull request Oct 2, 2018
Manually bring in PR 32 from original repo
APSL#32
tessus added a commit to tessus/react-native-version-info that referenced this pull request Oct 29, 2018
android {
compileSdkVersion 23
buildToolsVersion "23.0.1"
compileSdkVersion safeExtGet('compileSdkVersion', 26)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you change the version to 27?

compileSdkVersion 23
buildToolsVersion "23.0.1"
compileSdkVersion safeExtGet('compileSdkVersion', 26)
buildToolsVersion safeExtGet('buildToolsVersion', '26.0.3')
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you change the version to 27?

minSdkVersion 16
targetSdkVersion 22
minSdkVersion safeExtGet('minSdkVersion', 16)
targetSdkVersion safeExtGet('targetSdkVersion', 26)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you change the version to 27?

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.

2 participants