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

[DS-3795] bumped google-http-client-jackson2 to 1.23.0 #1909

Merged
merged 3 commits into from
Jan 23, 2018

Conversation

hardyoyo
Copy link
Member

@hardyoyo hardyoyo commented Jan 3, 2018

@tdonohue tdonohue added high priority code task Code cleanup task quick win Pull request is small in size & should be easy to review and/or merge labels Jan 16, 2018
@tdonohue tdonohue added this to the 7.0 milestone Jan 16, 2018
@terrywbrady terrywbrady self-requested a review January 19, 2018 17:00
@@ -1395,7 +1395,7 @@
<dependency>
<groupId>com.google.http-client</groupId>
<artifactId>google-http-client-jackson2</artifactId>
<version>1.21.0</version>
<version>1.23.0</version>
Copy link
Member

Choose a reason for hiding this comment

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

Looks good, but I think we should be syncing the version of google-http-client-jackson2 with the versions of google-http-client and google-api-client (at a minimum). I notice several of these Google dependencies (in the same region in this file) all are synced on the same 1.21.0 version, and I'm not sure that's an accident.

Copy link
Member

Choose a reason for hiding this comment

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

From the notes here, they all seem to have synced versions/releases: https://developers.google.com/api-client-library/java/

Copy link
Member Author

Choose a reason for hiding this comment

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

I agree, I've revised

@tdonohue tdonohue merged commit cecf494 into DSpace:master Jan 23, 2018
@hardyoyo hardyoyo deleted the DS-3795-bump-jackson-version branch January 24, 2019 03:49
@tdonohue tdonohue modified the milestones: 7.0, 7.0preview Jan 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code task Code cleanup task high priority quick win Pull request is small in size & should be easy to review and/or merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants