-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
TEP-0090: Add Matrix Package #4934
Conversation
cc @pritidesai |
e95dd6a
to
0ac390e
Compare
The following is the coverage report on the affected files.
|
0ac390e
to
74d2df3
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.
could you make it more clear in the commit message/PR description that these types are internal and won't be added to the API? i.e. no user facing changes
209b70a
to
1eff870
Compare
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: lbernick 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 |
1eff870
to
b765321
Compare
b765321
to
bfbe4a4
Compare
/lgtm |
/test pull-tekton-pipeline-integration-tests |
The following is the coverage report on the affected files.
|
/test pull-tekton-pipeline-integration-tests |
bfbe4a4
to
88fec99
Compare
The following is the coverage report on the affected files.
|
88fec99
to
8e99f73
Compare
The following is the coverage report on the affected files.
|
8e99f73
to
fc39863
Compare
The following is the coverage report on the affected files.
|
73c1474
to
721e517
Compare
The following is the coverage report on the affected files.
|
[TEP-0090][tep-0090] proposed executing a `PipelineTask` in parallel `TaskRuns` and `Runs` with substitutions from combinations of `Parameters` in a `Matrix`. In this change, we add the matrix package which generates the combinations of the `Parameters` in a `Matrix`. The types added are internal, and there are no user-facing changes in this PR. [tep-0090]: https://github.com/tektoncd/community/blob/main/teps/0090-matrix.md
721e517
to
9387e73
Compare
The following is the coverage report on the affected files.
|
/lgtm |
Changes
TEP-0090 proposed executing a
PipelineTask
in parallelTaskRuns
andRuns
with substitutions from combinations ofParameters
in aMatrix
.In this change, we add the matrix package which generates the combinations of the
Parameters
in aMatrix
.The types added are internal, and there are no user-facing changes in this PR.
/kind feature
Submitter Checklist
As the author of this PR, please check off the items in this checklist:
Release Notes