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

Surface via output if a backport PR has already been merged when trying to backport #280

Open
codebytere opened this issue Apr 11, 2024 · 1 comment
Labels
✨ enhancement New feature or request

Comments

@codebytere
Copy link
Member

As in title. Example: electron/electron#41793 showed needs-manual as a result of another bug but when I tried to run backports via trop again, it failed with blank diff instead of surfacing that the labels were just wrong and it'd already been merged into release branches: electron/electron#41805

@codebytere codebytere added the ✨ enhancement New feature or request label Apr 11, 2024
@dsanders11
Copy link
Member

Related to #252 - in general if any error happens the only thing visible to end users is a comment that it failed and a blank diff, so we need a general way to communicate an error occurred to end users instead of a confusing blank diff.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants