Skip to content
This repository has been archived by the owner on Jun 21, 2022. It is now read-only.

Update project to use Android Gradle plugin v3.x #55

Merged
merged 7 commits into from
Nov 3, 2017

Conversation

mustafa01ali
Copy link
Contributor

@mustafa01ali mustafa01ali commented Oct 27, 2017

  • Updated the project to use the latest Gradle plugin
  • Updated all the other dependencies
  • Updated Travis config
  • The response builder in OkHttp 3.8 has made the response message mandatory which Barricade was not passing, causing IllegalStateExceptions. I've fixed that issue by hardcoding a static message for now. In the future, we could make even the message configurable like we do for status codes.

Fixes #54

Updated the project to use the latest Gradle plugin and also bumped up all the other dependencies.

Also, the response builder in OkHttp 3.8 has made the message mandatory which Barricade was not passing, causing IllegalStateExceptions. I've fixed that issue by hardcoding a static message for now. In the future, we could make even the message configurable like we do for status codes.
@mustafa01ali
Copy link
Contributor Author

Alright, it's late now and Travis refuses to behave. Will look into the build failure tomorrow.

@Emeritus-DarranKelinske

thank you!

@Emeritus-DarranKelinske
Copy link

Does this help? OneBusAway/onebusaway-android@fa98297

@Emeritus-DarranKelinske
Copy link

This might be better: https://stackoverflow.com/a/28751112/3268303

@dazza5000
Copy link

@mustafa01ali i miss you

@mustafa01ali
Copy link
Contributor Author

Hey guys, got busy with work. Will try and work on this tonight.

@mustafa01ali
Copy link
Contributor Author

I spent several hours trying to fix the Travis emulator issue with no luck. A similar issue was reported to Travis back in March but they haven't bothered to look into it till now.

I've disabled running instrumentation tests for now to unblock this and created #56 to find a solution for the CI issue.

Note: Since these changes do not change the library API or behaviour, we will not be publishing an update to Bintray.

@mustafa01ali mustafa01ali merged commit b384c0c into master Nov 3, 2017
@mustafa01ali mustafa01ali deleted the adhoc/gradle-3-update branch November 3, 2017 05:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

issue when building project
3 participants