-
Notifications
You must be signed in to change notification settings - Fork 130
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Increase direct usage of Tekton Object Interface #896
Increase direct usage of Tekton Object Interface #896
Conversation
/assign @lcarva |
The following is the coverage report on the affected files.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
7714c5f
to
3a0b03f
Compare
The following is the coverage report on the affected files.
|
/retest |
3a0b03f
to
5741c0e
Compare
The following is the coverage report on the affected files.
|
The following is the coverage report on the affected files.
|
5741c0e
to
5693ab4
Compare
The following is the coverage report on the affected files.
|
5693ab4
to
14b715e
Compare
The following is the coverage report on the affected files.
|
14b715e
to
88aaba5
Compare
The following is the coverage report on the affected files.
|
The following is the coverage report on the affected files.
|
8b3fb31
to
b2eafed
Compare
The following is the coverage report on the affected files.
|
b2eafed
to
63223ad
Compare
The following is the coverage report on the affected files.
|
63223ad
to
cc9a743
Compare
/approve |
The following is the coverage report on the affected files.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Almost there! PR comment probably needs an update. 🙏
cc9a743
to
3228844
Compare
The following is the coverage report on the affected files.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Almost there!
This PR further increases the usage of the Tekton object interface in our codebase.
3228844
to
f68ca0f
Compare
The following is the coverage report on the affected files.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: chuangw6, wlynch 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:
Approvers can indicate their approval by writing |
This PR further increases the usage of the Tekton object interface in our codebase. It adds functions to get images from steps and sidecars and also extracts the
RefSource
fromStatus.Provenance
.Changes
Submitter Checklist
As the author of this PR, please check off the items in this checklist:
functionality, content, code)
Release Notes
/kind cleanup