search

HearthSim / Arcane-Tracker

An automatic Hearthstone tracker for Android
https://arcanetracker.com/
Other
148 stars 36 forks source link

Fix/Suppress lint errors #25

Closed williamwebb closed 8 years ago

williamwebb commented 8 years ago

The new dependencies are to fix an issue with lambdas & lint described here. The other changes here you should have been seeing. They are described here.

martinbonnin commented 8 years ago

How can I see these lambdas & lint errors ? My Android studio inspection seems fine (AS 2.2.2). If I run lint from the command line, I get a load of errors but did not see anything okio or lambda related.

I'm not against fixing lint warnings but I'd really like to get to the bottom of this as I haven't seen anything wrong from my side yet.

williamwebb commented 8 years ago

On you local run ~./gradlew clean build and it will run through all the lint tasks and generate a report. I will also create a MR with travis-ci integration for future use and it will show these errors.

martinbonnin commented 8 years ago

Ok, I have an error now. I would have expected this to happen also when building from Android Studio, not only from the command line and also when building assembleDebug and not just 'build'.

Anyway, PR merged, thanks !

martinbonnin commented 8 years ago

Also, it would be interesting to understand how doing ./gradlew app:lint differs from what Android Studio is doing to display all the yellow tooltips etc.