-
Notifications
You must be signed in to change notification settings - Fork 101
Conversation
…approche, isn't it? ;)
.travis.yml
Outdated
@@ -43,7 +43,7 @@ before_script: | |||
- adb shell input keyevent 82 & | |||
|
|||
script: | |||
- ./gradlew build jacocoTestReport assembleAndroidTest connectedCheck | |||
- ./gradlew clean build jacocoTestReport assembleAndroidTest connectedCheck |
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.
why clean? It will delete all build artifacts from the before_script
assemble
step
build.gradle
Outdated
assertjVersion = '2.5.0' | ||
mockitoVersion = '2.10.0' | ||
assertjVersion = '2.8.0' | ||
atslVersion = '1.0.1' |
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.
What does atsl
stand for?
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.
android test support library? I suggest supportTestVersion
supportLibraryVersion = '26.0.0' | ||
// compositeAndroidVersion should be always the same version as the supportLib version | ||
// Since composite is not up to date we can't use the latest one | ||
compositeAndroidVersion = supportLibraryVersion |
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.
good 👍
I found that the travis.yml also includes |
… a good approche, isn't it? ;)" This reverts commit bbe0ee6.
Yes the same comment should be added to the build tools version |
build.gradle
Outdated
|
||
supportLibraryVersion = '25.2.0' | ||
supportLibraryVersion = '26.0.0' |
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.
why not 26.1.0
. It includes LifecycleObserver which we could use for #109?
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.
Because of #115 (comment) 🙃
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.
Good point!
I've just updated all of our dependencies to the current latest version.