Skip to content
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.

Add coveralls.io #225

Merged
merged 1 commit into from
Jan 13, 2014
Merged

Add coveralls.io #225

merged 1 commit into from
Jan 13, 2014

Conversation

nschonni
Copy link
Contributor

Probably needs to be manually merged in with a new encrypted token in the .travis.yml

@andrew
Copy link
Contributor

andrew commented Jan 13, 2014

Why is the encrytped token required? I thought that was only for private repos/travis pro?

@nschonni
Copy link
Contributor Author

You may be right, I'll try that out

@andrew
Copy link
Contributor

andrew commented Jan 13, 2014

I couldn't find any good docs for it, where were you looking?

@nschonni
Copy link
Contributor Author

There was a page when looking at a repo that hasn't had coveralls setup that gave instructions, but I can't find a link to that set of docs. I do remember something about Travis-CI (not Pro) not requiring the .coveralls.yml and a bunch of other stuff.

@nschonni
Copy link
Contributor Author

I removed the token and it still worked! You'll probably need to enable the repo yourself on coveralls.io to the build to work correctly for PRs here. I'll squash the commits before this gets merged.

Adds JSCoverage instrumentation for coveralls.io on Travis
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling f9538c8 on nschonni:add-coveralls into * on andrew:master*.

@nschonni
Copy link
Contributor Author

Squash and pointed to this repo for the badge and config.

Running npm run-script coverage will still fail locally without the repo token, but I think that is OK.

@andrew
Copy link
Contributor

andrew commented Jan 13, 2014

Awesome, thanks!

andrew added a commit that referenced this pull request Jan 13, 2014
@andrew andrew merged commit 6898e80 into sass:master Jan 13, 2014
@nschonni nschonni deleted the add-coveralls branch January 13, 2014 20:14
jiongle1 pushed a commit to scantist-ossops-m2/node-sass that referenced this pull request Apr 7, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants