Skip to content

Don't return error from github provider on push event#372

Merged
smacker merged 1 commit intosrc-d:masterfrom
smacker:dont_return_error_on_push_post
Dec 3, 2018
Merged

Don't return error from github provider on push event#372
smacker merged 1 commit intosrc-d:masterfrom
smacker:dont_return_error_on_push_post

Conversation

@smacker
Copy link
Contributor

@smacker smacker commented Nov 28, 2018

Currently, we pollute logs with such "errors".
But it's not really an error we don't post anything on push by current design
(focus only on review and don't accommodate other github APIs).

Signed-off-by: Maxim Sukharev [email protected]

Currently we pollute logs with such "errors".
But it's not really an error we don't post anything on push by current design
(focus only on review and don't accomodate other github APIs).

Signed-off-by: Maxim Sukharev <[email protected]>
Copy link

@lookout-staging lookout-staging bot left a comment

Choose a reason for hiding this comment

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

If you have feedback about this comment, please, tell us.

Copy link
Contributor

@dpordomingo dpordomingo left a comment

Choose a reason for hiding this comment

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

🎉

Copy link
Contributor

@carlosms carlosms left a comment

Choose a reason for hiding this comment

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

👍

@smacker smacker merged commit 9d9dc66 into src-d:master Dec 3, 2018
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.

3 participants