Skip to content

Use latest knative/serving#149

Merged
google-prow-robot merged 2 commits intoknative:masterfrom
scothis:bump-serving
Jul 9, 2018
Merged

Use latest knative/serving#149
google-prow-robot merged 2 commits intoknative:masterfrom
scothis:bump-serving

Conversation

@scothis
Copy link
Copy Markdown
Contributor

@scothis scothis commented Jun 29, 2018

Now that there is an istio/v1alpha3 client in serving, we can drop our
client and share serving's client.

Refs #93 (comment)

/assign @mattmoor

Now that there is an istio/v1alpha3 client in serving, we can drop our
client and share serving's client.

Refs #93 (comment)
@google-prow-robot google-prow-robot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Jun 29, 2018
@mattmoor
Copy link
Copy Markdown
Member

This LGTM, but I'd like @tcnghia to confirm that there aren't any hot fixes coming :)

@mattmoor
Copy link
Copy Markdown
Member

/approve
/assign @tcnghia

I'll let @tcnghia give /lgtm

@google-prow-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mattmoor, scothis

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

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@google-prow-robot google-prow-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 29, 2018
@grantr
Copy link
Copy Markdown
Contributor

grantr commented Jul 2, 2018

After #138 goes in this will need to be rebased

@evankanderson
Copy link
Copy Markdown
Member

Oops, I just let #138 in. I should have kept it out in favor of this.

@grantr
Copy link
Copy Markdown
Contributor

grantr commented Jul 2, 2018

Maybe best to close this and create a new update PR when @tcnghia thinks we're ready.

@evankanderson
Copy link
Copy Markdown
Member

/lgtm

@google-prow-robot google-prow-robot added the lgtm Indicates that a PR is ready to be merged. label Jul 9, 2018
@google-prow-robot google-prow-robot merged commit 0f91b18 into knative:master Jul 9, 2018
@scothis scothis deleted the bump-serving branch July 9, 2018 17:03
matzew pushed a commit to matzew/eventing that referenced this pull request Jun 8, 2019
benmoss added a commit to benmoss/eventing that referenced this pull request Nov 10, 2021
Float the ko dependency, 0.6.0 is quite old and I think ko is stable
enough now
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants