-
Notifications
You must be signed in to change notification settings - Fork 140
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
GitHub Project Automation and Infra Updates #1168
Merged
rapids-bot
merged 12 commits into
nv-morpheus:branch-23.11
from
jarmak-nv:status-project-automations
Sep 8, 2023
Merged
GitHub Project Automation and Infra Updates #1168
rapids-bot
merged 12 commits into
nv-morpheus:branch-23.11
from
jarmak-nv:status-project-automations
Sep 8, 2023
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jarmak-nv
added
non-breaking
Non-breaking change
feature request
New feature or request
labels
Aug 31, 2023
Adding a do not merge label to this until I have a minute to add in fixes I'm making for CCCL around:
Done in a37cc6d |
Remove checkboxes from PR template and replace with Edit: Done in 65f54dc |
New issue to label for skip ci |
Add Edit: For brevity, I prefixed with |
mdemoret-nv
approved these changes
Sep 7, 2023
/merge |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Closes #1093
Closes #1109
This PR creates a suite of new GitHub project automations to make our lives easier, and updates the issue templates. For reference on the project automations, please see CCCL's workflows.
Issue Template Updates
bug_report.yml
print_env.sh
output to auto-hide/collapse for easier readingfeature_request.yml
Project Automation Updates
label_external_issues
I've updated the action to label external issues with both externalThis issue was filed by someone outside of the Morpheus team
and
Needs Triage
Need team to review and classify
. Doing so allows us to remove the
Needs Triage
Need team to review and classify
label but still track through Slack integrations on the
external
This issue was filed by someone outside of the Morpheus team
label.
Right now if we want it to continually be integrated in Slack it would need to keep Needs TriageNeed team to review and classify
in perpetuity which is inaccurate and makes the Triage view in the project messy. An issue we've put in the backlog is no longer in need of triage!
project_automation_set_in_progress
This action runs when a PR is opened, converted to a draft, or a review requests changes. It:
project_automation_set_in_review
This action runs when a PR is marked Ready for Review - promoting from a draft triggers this action. It:
project_automation_synchronize_issues
This action runs when a PR is edited, we're interested here because we should be editing in closing keywords for issues if new or old issues are realized to be closed by this PR. It:
project_automation_set_roadmap
This action runs on PR or Issue close. It:
project_automation_track_pr_dates
This action runs on PR open/close/ready for review. It:
When it can, it uses dates in the PR metadata, but a GHA doesn't actually have a date associated with it so we use opened/closed dates. For review, we format the date in a little shell command and store it in an environment variable, in EST.
Checklist
New or existing tests cover these changes.N/AThe documentation is up to date with these changes.N/A