Skip to content

Conversation

@dgerd
Copy link

@dgerd dgerd commented Nov 26, 2019

  • Reduce verbosity
  • Remove reference to closed issues
  • Remove reference to Knative Serving 0.3 upgrades
  • Use bash variable syntax in code snippets

* Reduce verbosity
* Remove reference to closed issues
* Remove reference to Knative Serving 0.3 upgrades
* Use bash variable syntax in code snippets
@googlebot googlebot added the cla: yes Indicates the PR's author has signed the CLA. label Nov 26, 2019
@knative-prow-robot knative-prow-robot added approved size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Nov 26, 2019
Copy link
Contributor

@RichieEscarez RichieEscarez left a comment

Choose a reason for hiding this comment

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

@dgerd if you are happy with my edit, I think this is ready to merge.

@RichieEscarez
Copy link
Contributor

FYI, I will commit one or two more changes to try and fix some funny rendering thats happening on the site: https://5de58052cff279fc5bf6e353--knative.netlify.com/docs/install/knative-custom-install/#installing-knative

@knative-prow-robot knative-prow-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Dec 3, 2019
@dgerd
Copy link
Author

dgerd commented Dec 3, 2019

I am good with it!

@knative-prow-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dgerd, RichieEscarez, zouyee

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

The pull request process is described here

Needs approval from an approver in each of these files:
  • OWNERS [RichieEscarez,dgerd]

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

@RichieEscarez RichieEscarez added the lgtm Indicates that a PR is ready to be merged. label Dec 3, 2019
@knative-prow-robot knative-prow-robot merged commit 5fc25bc into knative:master Dec 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes Indicates the PR's author has signed the CLA. lgtm Indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants