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.

Sign up for GitHub

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

Jump to bottom

Do not log exceptions when failures have been reported to the PR #69

Open
sbidoul opened this issue Aug 31, 2019 · 0 comments
Open

Do not log exceptions when failures have been reported to the PR #69

sbidoul opened this issue Aug 31, 2019 · 0 comments
Labels
Bot Task A task the bot does or should do enhancement New feature or request

Comments

@sbidoul
Copy link
Member

sbidoul commented Aug 31, 2019

Some merge command errors are reported both to the PR as a comment, and in the bot log (and in turn in sentry). This was made on purpose so bot administrators could spot them more easily.

Example: OCA/stock-logistics-warehouse#644 (comment)

Now that the bot is proving to be stable, it's not necessary to report such errors in the bot log anymore, to reduce notification noise.
@sbidoul sbidoul added enhancement New feature or request Bot Task A task the bot does or should do labels Aug 31, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bot Task A task the bot does or should do enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@sbidoul