-
Notifications
You must be signed in to change notification settings - Fork 31
update autovalue #103
update autovalue #103
Conversation
|
Fails on JDK 7: BUILD FAILED in 7s �[31;1mThe command "eval ./gradlew assemble " failed. Retrying, 3 of 3.�[0m
FAILURE: Build failed with an exception.
|
Codecov Report
@@ Coverage Diff @@
## master #103 +/- ##
=========================================
Coverage 60.48% 60.48%
Complexity 147 147
=========================================
Files 14 14
Lines 615 615
Branches 93 92 -1
=========================================
Hits 372 372
Misses 217 217
Partials 26 26Continue to review full report at Codecov.
|
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.
Ping ping ping
| compileOnly libraries.autovalue, | ||
| libraries.error_prone_annotations | ||
| compileOnly libraries.error_prone_annotations, | ||
| libraries.auto_value_annotations |
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.
I think auto_value is the compile only dependency and auto_value_annotations is an actual dependency
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.
done for annotation. auto_value should be on the annotationProcessor path.
build.gradle
Outdated
| codeGeneration libraries.auto_value, libraries.jsr305 | ||
| compile libraries.jsr305 | ||
| compileOnly libraries.autovalue | ||
| compileOnly libraries.auto_value_annotations |
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.
Same here
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.
done
* update autovalue * auto value requires Java 8 compiler * give auto_value_annotations compile scope
@chingor13
fixes #66
fixes #100