Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Turn off Coveralls PR bot commenting #2166

Closed
nschonni opened this issue Sep 2, 2016 · 6 comments
Closed

Turn off Coveralls PR bot commenting #2166

nschonni opened this issue Sep 2, 2016 · 6 comments

Comments

@nschonni
Copy link
Collaborator

nschonni commented Sep 2, 2016 •
edited by HamptonMakes
Loading

It's probably just a matter of preference, but since i started watching the repo again, i noticed alot of emails from Coveralls. In the node-sass repo we have the hook so it still shows up in the PR status, but doesn't have the bot comment https://coveralls.zendesk.com/hc/en-us/articles/201774895-Notifications
@xzyfer
Copy link
Contributor

xzyfer commented Sep 12, 2016

Make it so

@nschonni
Copy link
Collaborator Author

@xzyfer you'll need to do it since you need repo admin to change the hooks

@xzyfer
Copy link
Contributor

xzyfer commented Sep 13, 2016

I think I flicked the right switch. The coveralls UI isn't obvious.

@xzyfer xzyfer closed this as completed Sep 13, 2016
@nschonni
Copy link
Collaborator Author

Looks like this came back again. Maybe it is also setup by some of the other maintainers like we had with the Appveyor duplicates

@xzyfer
Copy link
Contributor

xzyfer commented Sep 22, 2016

Ok I think I have it this time for reals.

@xzyfer
Copy link
Contributor

xzyfer commented Sep 22, 2016

Huzzah! #2184

# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@xzyfer @nschonni