Skip to content
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

Have release script open Gutenberg PR #2523

Merged
merged 5 commits into from
Aug 3, 2020
Merged

Conversation

mchowning
Copy link
Contributor

This adds opening the gutenberg release PR to the release automation script. In doing that, it automatically includes a link to the gb-mobile PR in the gutenberg PR description. In addition, it prints the list of opened PRs at the end of the script.

Assuming this works well, I think it should be relatively straightforward to add the WPiOS and WPAndroid PRs to this script as well. But I figured we should see how this works first.

To test:
Make sure the release script runs smoothly and opens draft PRs in gb && gb-mobile with appropriately updated branches.

PR submission checklist:

  • I have considered adding unit tests where possible.
  • I have considered if this change warrants user-facing release notes more info and have added them to RELEASE-NOTES.txt if necessary.

Copy link
Contributor

@ceyhun ceyhun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks amazing 🎉 Thank you so much @mchowning! Just have a small suggestion for better readability. Can't wait to try it!

bin/release_automation.sh Outdated Show resolved Hide resolved
bin/release_automation.sh Outdated Show resolved Hide resolved
Co-authored-by: Ceyhun Ozugur <ceyhunozugur@gmail.com>
@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Aug 3, 2020

Wanna run full suite of Android and iOS UI tests? Click here and 'Approve' CI job!

Co-authored-by: Ceyhun Ozugur <ceyhunozugur@gmail.com>
@mchowning
Copy link
Contributor Author

Thanks for the suggestions @ceyhun! I do like how that makes it much clearer what the script is doing.

@mchowning mchowning merged commit edb292a into develop Aug 3, 2020
@mchowning mchowning deleted the add_gb_pr_to_release_script branch August 3, 2020 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants