Skip to content

Added travis and coveralls badges to README#6

Merged
majames merged 1 commit intomasterfrom
chore/coverage-stats
May 24, 2017
Merged

Added travis and coveralls badges to README#6
majames merged 1 commit intomasterfrom
chore/coverage-stats

Conversation

@majames
Copy link
Copy Markdown
Collaborator

@majames majames commented May 24, 2017

🎫

@majames majames requested a review from zzarcon May 24, 2017 09:38
@coveralls
Copy link
Copy Markdown

Coverage Status

Changes Unknown when pulling 067314c257013d723629fe06ca0763e99752952e on chore/coverage-stats into ** on master**.

@majames majames force-pushed the chore/coverage-stats branch from 067314c to 9696cf1 Compare May 24, 2017 09:40
@coveralls
Copy link
Copy Markdown

Coverage Status

Changes Unknown when pulling 9696cf1fd839561359dcb7455a4d8bff77e8b440 on chore/coverage-stats into ** on master**.

Comment thread .travis.yml Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can we wrap this into a package script and just yarn coverage later?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

The command will only work when run within Travis so I didn't want to add it as a script. I'll add a yarn coverage command that makes sense for devs to run locally doe.

@zzarcon
Copy link
Copy Markdown
Member

zzarcon commented May 24, 2017

I think the coveralls bot is too noisy, an we avoid somehow the comments on the PRs? what do you think?

@majames
Copy link
Copy Markdown
Collaborator Author

majames commented May 24, 2017

disabled 💬 🙉 ⛔️

@majames majames force-pushed the chore/coverage-stats branch from 9696cf1 to 3a407ff Compare May 24, 2017 22:01
@majames majames merged commit 893e7d1 into master May 24, 2017
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