Skip to content

Remove coverage.sh, it's not used anymore#1634

Merged
adrcunha merged 1 commit intoknative:masterfrom
adrcunha:bye-old-script
Jul 23, 2018
Merged

Remove coverage.sh, it's not used anymore#1634
adrcunha merged 1 commit intoknative:masterfrom
adrcunha:bye-old-script

Conversation

@adrcunha
Copy link
Copy Markdown
Contributor

Code coverage is now handled by a hermetic Prow job.

@google-prow-robot google-prow-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jul 20, 2018
@adrcunha adrcunha requested review from mattmoor and removed request for vaikas July 20, 2018 17:23
@steuhs
Copy link
Copy Markdown
Contributor

steuhs commented Jul 20, 2018

/approve

@google-prow-robot google-prow-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jul 23, 2018
@knative knative deleted a comment from google-prow-robot Jul 23, 2018
@knative knative deleted a comment from google-prow-robot Jul 23, 2018
@srinivashegde86
Copy link
Copy Markdown
Contributor

/lgtm

@google-prow-robot google-prow-robot added the lgtm Indicates that a PR is ready to be merged. label Jul 23, 2018
@adrcunha adrcunha merged commit d29fb1e into knative:master Jul 23, 2018
@adrcunha adrcunha deleted the bye-old-script branch July 23, 2018 17:44
bbrowning pushed a commit to bbrowning/serving that referenced this pull request Oct 5, 2018
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/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants