Giters

wordpress-mobile / release-toolkit-gutenberg-mobile

Automation Scripts for Releasing Gutenberg-Mobile Updates to the WordPress Mobile Apps.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

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

derekblank opened this issue · comments

commented

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.

commented

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