-
Notifications
You must be signed in to change notification settings - Fork 110
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
issue template updates #824
issue template updates #824
Conversation
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.
This looks good so far, minor feedback and some questions about how we want to approach some of the implied consequences. Otherwise, very good stuff and I hope to merge soon.
Sorry for the delayed review, @brian-ruf. Also that feature branch isn't necessary and I wasn't fast enough to get rid of it. We can just target master for this given the other issues I need to report. I will take care of that bug report soonish. 😄 |
Co-authored-by: A.J. Stein <[email protected]>
Co-authored-by: A.J. Stein <[email protected]>
Co-authored-by: A.J. Stein <[email protected]>
Co-authored-by: A.J. Stein <[email protected]>
Co-authored-by: A.J. Stein <[email protected]>
Co-authored-by: A.J. Stein <[email protected]>
Co-authored-by: A.J. Stein <[email protected]>
Co-authored-by: A.J. Stein <[email protected]>
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.
This looks great @brian-ruf. I'll add my approval once the comments @aj-stein-gsa sent are resolved
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.
Sorry I held this up for longer in the day I had planned, I like it from Friday and you addressed all the feedback I was concerned with. The other feedback items are not critical.
(Thanks to @Rene2mt to remind me how tardy I am this late in the afternoon.)
* issue template updates * fixed attributes * Update .github/ISSUE_TEMPLATE/4-constraint-work.yaml Co-authored-by: A.J. Stein <[email protected]> * Update .github/ISSUE_TEMPLATE/3-action-item.yaml Co-authored-by: A.J. Stein <[email protected]> * Update .github/ISSUE_TEMPLATE/4-constraint-work.yaml Co-authored-by: A.J. Stein <[email protected]> * Update .github/ISSUE_TEMPLATE/4-constraint-work.yaml Co-authored-by: A.J. Stein <[email protected]> * Update .github/ISSUE_TEMPLATE/4-constraint-work.yaml Co-authored-by: A.J. Stein <[email protected]> * Update .github/ISSUE_TEMPLATE/4-constraint-work.yaml Co-authored-by: A.J. Stein <[email protected]> * Update .github/ISSUE_TEMPLATE/4-constraint-work.yaml Co-authored-by: A.J. Stein <[email protected]> * Update .github/ISSUE_TEMPLATE/4-constraint-work.yaml Co-authored-by: A.J. Stein <[email protected]> --------- Co-authored-by: A.J. Stein <[email protected]>
Committer Notes
This PR adds one new issue template and modifies an existing issue template.
The new issue template is specifically for constraint work.
The action-item template was modified to remove obsolete choices and to direct the submitter to the automate.fedramp.gov repo for issues related to documentation.
All Submissions:
By submitting a pull request, you are agreeing to provide this contribution under the CC0 1.0 Universal public domain dedication.