-
Notifications
You must be signed in to change notification settings - Fork 38.6k
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
Use github forms for issue templates #104468
Conversation
lgtm |
Ah...sorry, this slipped through the cracks for me. Will review shortly 👍 |
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.
Thanks! This has actually been on my todo for a bit, but fell into the realm of E_TOO_MANY_THINGS.
I have a few small suggestions, and after that it should be good to go :)
/triage accepted |
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.
Suggested changes based on #105190 🙂
Also, it won't be possible to use prow commands from the form entries right? So for example, if someone entered /sig node
in the form, would prow be able to apply the correct label?
15985e3
to
4fe2fdc
Compare
You can see the results here: https://github.com/thockin/kubernetes/issues/new/choose
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.
PTAL
They will come through, when posted its all markdown and prow responds to the content of the message |
/lgtm |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mrbobbytables, thockin 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 changed from .md to .yaml in kubernetes/kubernetes#104468.
You can see the results here: https://github.com/thockin/kubernetes/issues/new/choose
Sadly it doesn't seem to work for PRs.
/kind cleanup