Java 11: bring jacoco-maven-plugin up to current - Githubissues

sendgrid / java-http-client

SendGrid's Java HTTP Client for calling APIs

https://sendgrid.com

MIT License

25 stars 78 forks source link

Java 11: bring jacoco-maven-plugin up to current #119

Closed kebeda closed 4 years ago

kebeda commented 5 years ago

Fixes #118

Checklist

[x] I have made a material change to the repo (functionality, testing, spelling, grammar)
[x] I have read the [Contribution Guide] and my PR follows them.
[x] I updated my branch with the master branch.
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation about the functionality in the appropriate .md file
[ ] I have added in line documentation to the code I modified

Short description of what this PR does:

This change will allow mvn test to succeed when run under a Java 11 JVM/javac.

It resolves it by updating the jacoco-maven-plugin to a version that can run in this configuration.

kebeda commented 5 years ago

Didn't check the last 3 checkboxes on the PR description because,

The existing tests cover this
No user-visible changes are in this PR
There was no code changed, hence no comment needed