search

wordpress-mobile / release-toolkit-gutenberg-mobile

Automation Scripts for Releasing Gutenberg-Mobile Updates to the WordPress Mobile Apps.
Mozilla Public License 2.0
5 stars 2 forks source link

[CLI] Update `integrate` command output when PRs already exist #244

Open derekblank opened 6 months ago

derekblank commented 6 months ago

I noted that when running the integrate command a second time to change the Gutenberg ref from a commit back to a tag, the output could be updated:

[INFO] PR already exists: https://github.com/wordpress-mobile/WordPress-iOS/pull/22513
[INFO] Created PR https://github.com/wordpress-mobile/WordPress-Android/pull/20093
[INFO] Created PR https://github.com/wordpress-mobile/WordPress-iOS/pull/22513

For iOS, it notes that the PR already exists, but not for Android. For both platforms it says "Created PR", but perhaps we should just say "Updated PR" followed by the link to avoid confusion.

The existing PRs are updated correctly and no functionality is broken, so this is just a minor command output issue.

derekblank commented 6 months ago

@jhnstn If this is already covered in another issue, we can close this one. I didn't see one, however.