-
Notifications
You must be signed in to change notification settings - Fork 68
Version 4.17.0 #49
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
Version 4.17.0 #49
Conversation
android { | ||
compileSdkVersion 26 | ||
buildToolsVersion "26.0.3" | ||
compileSdkVersion rootProject.ext.compileSdkVersion |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's cool!
project(':react-native-adjust').projectDir = new File(rootProject.projectDir, '../node_modules/react-native-adjust/android') | ||
include ':react-native-adjust' | ||
project(':react-native-adjust').projectDir = new File(rootProject.projectDir, '../node_modules/react-native-adjust/android') | ||
include ':react-native-adjust' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
duplicated instruction
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, build scripts which are building and running our example app are making sure this happens each time we run them, but having in mind that it's an example app that we mostly use just to demonstrate how SDK API can be used, we're living with it. Will be addressed in future, but for now, it's like this. But fair point. 👍
getSdkVersion()
method toAdjust
interface to obtain current SDK version string.compile
keyword in plugin'sbuild.gradle
file intoimplementation
(WARNING: Configuration 'compile' is obsolete and has been replaced with 'implementation' and 'api' #47). Thanks to @vitorreis.