Skip to content
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

[WFLY-13086] Upgrade gson from 2.8.2 to 2.8.6 #12987

Merged
merged 1 commit into from Mar 2, 2020

Conversation

spyrkob
Copy link
Contributor

@spyrkob spyrkob commented Feb 13, 2020

Issue: https://issues.redhat.com/browse/WFLY-13086

Downstream dependency to align with Runtimes versions

Changes: google/gson@gson-parent-2.8.2...gson-parent-2.8.5

@spyrkob
Copy link
Contributor Author

spyrkob commented Feb 13, 2020

@ehsavoie could you review please?

@wildfly-ci wildfly-ci added the deps-ok Dependencies have been checked, and there are no significant changes label Feb 13, 2020
@ehsavoie
Copy link
Contributor

Jaeger client is on 2.8.6. Why not going all the way to 2.8.6 ?

@spyrkob
Copy link
Contributor Author

spyrkob commented Feb 13, 2020

@ehsavoie I was going by downstream version, if it's OK to go to most recent version I can update the PR

@bstansberry
Copy link
Contributor

@spyrkob Please go to 2.8.6. The only use of this lib is Jaeger and the pom of the 0.34.3 version we use specifies 2.8.6 so WF should match.

@spyrkob spyrkob changed the title [WFLY-13086] Upgrade gson from 2.8.2 to 2.8.5 [WFLY-13086] Upgrade gson from 2.8.2 to 2.8.6 Feb 20, 2020
@bstansberry bstansberry merged commit 7743785 into wildfly:master Mar 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deps-ok Dependencies have been checked, and there are no significant changes
Projects
None yet
4 participants