Closed sayan-biswas closed 11 months ago
The following is the coverage report on the affected files.
Say /test pull-tekton-results-go-coverage
to re-run this coverage report
File | Old Coverage | New Coverage | Delta |
---|---|---|---|
pkg/watcher/reconciler/dynamic/dynamic.go | 66.9% | 68.3% | 1.4 |
/test pull-tekton-results-build-tests
The following is the coverage report on the affected files.
Say /test pull-tekton-results-go-coverage
to re-run this coverage report
File | Old Coverage | New Coverage | Delta |
---|---|---|---|
pkg/watcher/reconciler/dynamic/dynamic.go | 66.9% | 68.3% | 1.4 |
The following is the coverage report on the affected files.
Say /test pull-tekton-results-go-coverage
to re-run this coverage report
File | Old Coverage | New Coverage | Delta |
---|---|---|---|
pkg/watcher/reconciler/dynamic/dynamic.go | 66.9% | 68.3% | 1.4 |
@Roming22: changing LGTM is restricted to collaborators
/approve /lgtm on behalf of @Roming22
/approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: avinal, enarha, Roming22
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/lgtm
As per discussion in #648
Changes
This PR makes the owner reference check in watcher optional.
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you review them:
/kind <type>
. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tepRelease Notes