-
Notifications
You must be signed in to change notification settings - Fork 267
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
Initial pre-submit script… #72
Initial pre-submit script… #72
Conversation
8957414
to
d54a035
Compare
… partly imported from `tektoncd/pipeline`. This is a really early draft but should allow better test coverage on dashboard to start. Signed-off-by: Vincent Demeester <[email protected]>
d54a035
to
5f2859e
Compare
Race condition issue : #73 |
/hold cancel |
oups sorry @abayer =) /assign @a-roberts |
Output from this looks great and any improvements can be made after - thank you so much @vdemeester for working on this! /lgtm /approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: a-roberts, vdemeester 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 |
/lgtm (poking Prow) |
… partly imported from
tektoncd/pipeline
/hold
Signed-off-by: Vincent Demeester [email protected]