feat: Further Error Messaging

tibdex / backport

🔙 GitHub Action to backport pull requests

MIT License

73 stars 65 forks source link

feat: Further Error Messaging #85

Closed therynamo closed 2 years ago

therynamo commented 2 years ago

When using backport we found two things in relation to error messaging:

The author wasn't aware the backport failed
The worktree commands weren't always a reliable way to resolve the issue

In order to rectify these "issues" we modified the error messaging to include:

Mention the author for a Pull Request at the beginning of the error messaging
Add a section for a more rudimentary cherry-pick without work trees

tibdex commented 2 years ago

Mention the author for a Pull Request at the beginning of the error messaging

I think it would make more sense to notify the user who added the backport label to the PR. Anyway, I would prefer not adding this behavior at all and let users configure their notifications as they want instead.

The worktree commands weren't always a reliable way to resolve the issue

What was the issue? I don't want to offer two ways to resolve the issue when a single one can suffice.