-
Notifications
You must be signed in to change notification settings - Fork 664
DYN-9484 Update Action Title Body #16551
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
DYN-9484 Update Action Title Body #16551
Conversation
For sanity check I'm modifying the PR title and also in the body we included the eidos team.
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.
See the ticket for this pull request: https://jira.autodesk.com/browse/DYN-9484
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.
Pull Request Overview
This PR updates the automated GitHub Action workflow for HTML documentation updates by modifying the pull request title format and body content. The changes add a standardized title prefix and include the eidos team in the PR body for review notifications.
- Adds "DYN-000 -" prefix to the automated PR title
- Updates the PR body to mention @DynamoDS/eidos team for review
gh pr create --title "DYN-000 - Update HTML Documentation" \ | ||
--body "This PR contains automated changes generated due to changes in doc/distrib/html subfolders, @DynamoDS/eidos, please review and merge this PR" \ | ||
--base $MAIN_BRANCH \ | ||
--head ${{ steps.commit_changes.outputs.branch }} |
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.
can we use --assignee
flag as well to assign the PR to the team?
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.
added in the next commit feb642a
run: | | ||
gh pr create --title "Update HTML Documentation" \ | ||
--body "This PR contains automated changes generated due to changes in doc/distrib/html subfolders" \ | ||
gh pr create --title "DYN-000 - Update HTML Documentation" \ |
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.
I think we can use DYN-9484 instead, that way we can see the PR in jira as well.
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.
updated in the next commit feb642a
Updated the title and also added assignee
Purpose
For sanity check I'm modifying the PR title and also in the body we included the eidos team.
Declarations
Check these if you believe they are true
Release Notes
For sanity check I'm modifying the PR title and also in the body we included the eidos team.
Reviewers
@avidit
FYIs
@zeusongit