add workflow description section to the template #1090
Conversation
openshift-ci
bot
added
the
do-not-merge/work-in-progress
openshift-ci
bot
requested review from
aravindhp,
bparees,
kikisdeliveryservice,
imcleod and
markmc
guidelines/enhancement_template.md
Outdated
|
|
||
| See | ||
| https://github.com/openshift/enhancements/blob/master/enhancements/workload-partitioning/management-workload-partitioning.md#high-level-end-to-end-workflow | ||
| and https://github.com/openshift/enhancements/pull/1067 for more detailed examples. |
There was a problem hiding this comment.
I'll keep this PR a WIP until the second example here is merged and I can link to the published version of the doc.
dhellmann
force-pushed
the
template-workflow-section
branch
from
2c4f9bf
to
1880c87
Compare
dhellmann
force-pushed
the
template-workflow-section
branch
from
436ad6f
to
4915dea
Compare
I would be surprised to find us making changes for which the user will neither provide input nor see any impact. Do you have an example in mind? |
dhellmann
force-pushed
the
template-workflow-section
branch
from
4915dea
to
f645959
Compare
dhellmann
removed
the
do-not-merge/work-in-progress
|
Rebased and removed the WIP label |
dhellmann
changed the title
|
/hold We still want to wait on this to update it after the linked example in #1067 is merged. |
openshift-ci
bot
added
the
do-not-merge/hold
dhellmann
force-pushed
the
template-workflow-section
branch
from
f645959
to
74726b1
Compare
dhellmann
force-pushed
the
template-workflow-section
branch
from
74726b1
to
721433b
Compare
Be explicit about the expectation that user stories are written using a formula. Signed-off-by: Doug Hellmann <dhellmann@redhat.com>
Add a new section to the enhancement template to encourage enhancement authors to explain how someone will use the feature they are describing. Signed-off-by: Doug Hellmann <dhellmann@redhat.com>
dhellmann
force-pushed
the
template-workflow-section
branch
from
721433b
to
2d69b08
Compare
|
/hold cancel |
openshift-ci
bot
removed
the
do-not-merge/hold
|
@bparees @kikisdeliveryservice this should be ready to go now |
|
/approve |
openshift-ci
bot
added
the
do-not-merge/hold
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bparees 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 |
openshift-ci
bot
added
the
approved
|
/hold cancel Let's move along. We can tweak further if we need to. |
openshift-ci
bot
removed
the
do-not-merge/hold
|
@dhellmann: all tests passed! Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
Add a new section to the enhancement template to encourage enhancement
authors to explain how someone will use the feature they are
describing.