-
Notifications
You must be signed in to change notification settings - Fork 163
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
Create PRs on other repos with GitHub CLI #1765
Create PRs on other repos with GitHub CLI #1765
Conversation
16b993f
to
9a43f17
Compare
fc4417f
to
f3de41f
Compare
Removes the warnings from the workflow summary page. <https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/>
Job summaries are more visible than log messages from a step.
c510792
to
1fcc625
Compare
1fcc625
to
8505757
Compare
No functional change here except removal of the PR number from the job summary (URL is more directly available and useful).
8505757
to
c448302
Compare
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.
Nice use of the action summary! Changes look good to me 👍
@@ -81,10 +81,17 @@ jobs: | |||
This will surface any issues that can arise from merging the PR in Auspice. To address these issues locally, update the source branch (e.g. with a git rebase). | |||
|
|||
This message and corresponding commits were automatically created by a GitHub Action from [nextstrain/auspice](https://github.com/nextstrain/auspice). | |||
draft: true | |||
delete-branch: true |
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.
non-blocking
I don't see an equivalence for the delete-branch
input, but I'm not sure we even used it as intended?
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.
From the action's README:
If a pull request exists and new changes on the base branch make the pull request unnecessary (i.e. there is no longer a diff between the pull request branch and the base), the pull request is automatically closed. Additionally, if
delete-branch
is set totrue
thebranch
will be deleted.
Yeah, I don't think we actually used it - we just close the PRs and delete the branches manually, which is fine.
Very nice to see! |
Description of proposed changes
Motivation: move away from 3rd party Actions.
No functional change here except removal of the PR number from outputs (URL is more directly available and useful)
Related issue(s)
Checklist
If making user-facing changes, add a message in CHANGELOG.md summarizing the changes in this PRno functional changespeter-evans/create-pull-request
from allow list in GitHub org settings (it was only used in this repo)