Skip to content
This repository has been archived by the owner on Jul 19, 2019. It is now read-only.

Add new dependency: com.squareup.okhttp3:okhttp #34

Merged
merged 1 commit into from
Mar 13, 2017

Conversation

snjeza
Copy link

@snjeza snjeza commented Mar 8, 2017

Add new dependency: com.squareup.okhttp3:okhttp - eclipse-che/che#4363

Copy link

@amisevsk amisevsk left a comment

Choose a reason for hiding this comment

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

LGTM. 3.4.2 is the version used in kubernetes-client 2.1.1.

pom.xml Outdated
@@ -45,6 +45,7 @@
<com.googlecode.gson.version>2.3.1</com.googlecode.gson.version>
<com.h2database.version>1.4.192</com.h2database.version>
<com.jcraft.jsch.version>0.1.53</com.jcraft.jsch.version>
<com.squareup.okhttp3>3.4.2</com.squareup.okhttp3>
Copy link

Choose a reason for hiding this comment

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

Could we use com.square.okhttp3.version to match the other variables?

@amisevsk amisevsk merged commit 58b4743 into eclipse-che:openshift-connector Mar 13, 2017
@skabashnyuk
Copy link
Contributor

skabashnyuk commented Mar 13, 2017

@amisevsk can you make sure that
if this transitive dependency is not used or approved over CQ

<groupId>com.squareup.okio</groupId>
<artifactId>okio</artifactId>
</dependency>

@amisevsk
Copy link

@skabashnyuk We are currently updating dependencies in the openshift-connector branch to avoid having to go through the CQ process too frequently. As Che issue #2847 progresses, we'll probably have to update these same dependencies again (e.g. if we need to move to a newer version of kubernetes-client).

Once we open a PR to merge the openshift-connector branch into master, we'll go through and ensure CQs are opened on all updated transitive dependencies.

@skabashnyuk
Copy link
Contributor

ok

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants