-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merge 19.xfixes #32
Merge 19.xfixes #32
Conversation
e3123b2
to
4af4f5b
Compare
LGTM , one more thing, if possible, to include :
|
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.
one more thing, if possible, to include :
- upgrade jacoco maven plugin version to
0.8.6
- tested with: https://github.com/gravitee-io/gravitee-common/blob/d1df642d266a4537a1160593f582bad26a2f607e/pom.xml#L36
- that's for QA
<jersey.version>2.30.1</jersey.version> | ||
<logback.version>1.2.3</logback.version> | ||
<mockito.version>2.24.5</mockito.version> | ||
<mockito.version>3.6.0</mockito.version> | ||
<jackson.version>2.10.3</jackson.version> | ||
<rxjava.version>2.2.19</rxjava.version> | ||
<reactive-streams.version>1.0.3</reactive-streams.version> | ||
<jacoco-maven-plugin.version>0.7.7.201606060606</jacoco-maven-plugin.version> |
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.
one more thing, if possible, to include :
- upgrade jacoco maven plugin version to
0.8.6
- tested with: https://github.com/gravitee-io/gravitee-common/blob/d1df642d266a4537a1160593f582bad26a2f607e/pom.xml#L36
- that's for QA
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.
You pointed the mockito version but talk about jacoco version. Is it a mistake ?
Anyway, I would prefer to handle the jacoco upgrade in a dedicated PR as it is not related to this one (which is about merging the branch 19.xfixes into master).
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.
Sry about the mmispointed link ... tyhe part I meant to point at is <jacoco-maven-plugin.version>0.8.6</jacoco-maven-plugin.version>
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.
ok lert's do jacoco upgrade separately
…ee repos which had a gravitee-parent 19
…y changing the logback configuration Closes gravitee-io/issues#5139
4af4f5b
to
1ba129b
Compare
No description provided.