Skip to content

Conversation

RandomByte
Copy link
Member

@RandomByte RandomByte commented Dec 17, 2018

Ignore push builds. This should reduce the number of comments Coveralls adds to PRs

@coveralls
Copy link

coveralls commented Dec 17, 2018

Coverage Status

Coverage remained the same at 84.261% when pulling 32710a2 on coveralls-pr-only into 989c935 on master.

@RandomByte RandomByte requested a review from matz3 December 17, 2018 18:53
@RandomByte RandomByte force-pushed the coveralls-pr-only branch 2 times, most recently from 3c5ec7f to 5b2d437 Compare December 18, 2018 13:54
matz3
matz3 previously approved these changes Dec 19, 2018
Ignore push builds. This should reduce the number of comments
Coveralls adds to PRs
On test failures, npms error logs distorted the nyan cat rendering
@RandomByte RandomByte changed the title [INTERNAL] Coveralls: Only send PR build result Coveralls: Only send PR build result Dec 19, 2018
@RandomByte RandomByte merged commit d6c9f81 into master Dec 20, 2018
@RandomByte RandomByte deleted the coveralls-pr-only branch December 20, 2018 13:16
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