Skip to content

update to gson 2.10#689

Merged
cdietrich merged 1 commit into
mainfrom
cd_gson210
Jan 16, 2023
Merged

update to gson 2.10#689
cdietrich merged 1 commit into
mainfrom
cd_gson210

Conversation

@cdietrich

Copy link
Copy Markdown
Contributor

update to gson 2.10

@cdietrich cdietrich added this to the 0.20.0 milestone Jan 2, 2023
Signed-off-by: Christian Dietrich <christian.dietrich@itemis.de>
Comment thread gradle/versions.gradle
'guava_orbit': '30.1.0.v20221112-0806',
'gson': '[2.9.1,2.10)',
'gson_orbit': '2.9.1.v20220915-1632',
'gson': '[2.10,2.11)',

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
'gson': '[2.10,2.11)',
'gson': '[2.9.1,2.11)',

Gson 2.10 just added new methods (see eclipse-m2e/m2e-core#1146 (comment)). So if lsp4j is using gson as consumer and not as provider, the lower can actually stay where it is.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dont know how versioning works in gson and if using upper range will pick new version in gradle and thus a new method may accidentially be used

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

at least gradle build scan shows new version picked
https://scans.gradle.com/s/yudurt3xbdgie/dependencies?dependencies=gson&expandAll
am not sure if the 2.10.1 will also drop to orbit.
dont know if and when i will find time to bump

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://git.eclipse.org/r/c/orbit/orbit-recipes/+/199278
but that one will come with m2 only

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

=> propose to merge this one and then do another pr for m2

@cdietrich cdietrich merged commit 8746945 into main Jan 16, 2023
@cdietrich cdietrich deleted the cd_gson210 branch January 16, 2023 18:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants