Skip to content

Skip updating dependency file if it is not specified#730

Merged
istio-merge-robot merged 1 commit intomasterfrom
rkpagadala-patch-5
Mar 8, 2018
Merged

Skip updating dependency file if it is not specified#730
istio-merge-robot merged 1 commit intomasterfrom
rkpagadala-patch-5

Conversation

@rkpagadala
Copy link
Copy Markdown
Contributor

this allows us to just update istio.deps with new shas and not have any other dependencies, used for PROXY_TAG in istio repo

@googlebot googlebot added the cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. label Mar 8, 2018
@rkpagadala rkpagadala requested review from bochunz, chxchx and sebastienvas and removed request for bochunz March 8, 2018 02:12
@chxchx
Copy link
Copy Markdown
Contributor

chxchx commented Mar 8, 2018

/lgtm

@istio-merge-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: chxchx

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these OWNERS Files:

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

@istio-merge-robot
Copy link
Copy Markdown

/test all [submit-queue is verifying that this PR is safe to merge]

@istio-merge-robot
Copy link
Copy Markdown

Automatic merge from submit-queue.

@istio-merge-robot istio-merge-robot merged commit ec534ca into master Mar 8, 2018
@yutongz yutongz deleted the rkpagadala-patch-5 branch March 9, 2018 01:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants