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

Added notification on error #9

Merged
merged 6 commits into from
Feb 5, 2024
Merged

Added notification on error #9

merged 6 commits into from
Feb 5, 2024

Conversation

Bullrich
Copy link
Collaborator

@Bullrich Bullrich commented Feb 5, 2024

Added the ability to comment in a PR when it fails to update it.

It will link to the logs and inform the user on what they can do.

This resolves #8

@Bullrich Bullrich requested a review from a team as a code owner February 5, 2024 10:44
@Bullrich Bullrich linked an issue Feb 5, 2024 that may be closed by this pull request
@Bullrich
Copy link
Collaborator Author

Bullrich commented Feb 5, 2024

Example log message:

Failed to update PR ❌

There was an error while trying to keep this PR up-to-date

You may have conflicts ‼️ or may have to manually sync it with the target branch 👉❇️

More info in the logs 📋

Copy link
Contributor

@mordamax mordamax left a comment

Choose a reason for hiding this comment

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

🚀

@Bullrich Bullrich merged commit faa840c into main Feb 5, 2024
8 checks passed
@Bullrich Bullrich deleted the notify-on-error branch February 5, 2024 12:05
fellowship-merge-bot bot pushed a commit to polkadot-fellows/runtimes that referenced this pull request Feb 14, 2024
Upgraded the action
[paritytech/up-to-date-action](https://github.com/paritytech/up-to-date-action)
to version `0.2.0`.

This version brings paritytech/up-to-date-action#9. A feature that, when
the action fails with a PR, it comments in it so the user can try to fix
the problem instead of silently failing.

- [x] Does not require a CHANGELOG entry
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Report on PR when it failed to update it
2 participants