Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Remove coverage reporting with Coveralls #2494

Merged
merged 1 commit into from
Jun 9, 2014
Merged

Remove coverage reporting with Coveralls #2494

merged 1 commit into from
Jun 9, 2014

Conversation

doktorbro
Copy link
Member

After 9 months of usage it wasn’t a great helper. The service looks dead to me.

@parkr
Copy link
Member

parkr commented Jun 9, 2014

Lolz, we're featured on their homepage.

screen shot 2014-06-08 at 9 53 04 pm

@doktorbro
Copy link
Member Author

Haha, I noticed it too. The badge there points to the old mojombo namespace. The homepage seems to be static.

@parkr parkr merged commit b7d7f6d into jekyll:master Jun 9, 2014
@parkr
Copy link
Member

parkr commented Jun 9, 2014

Bye, Coveralls!

parkr added a commit that referenced this pull request Jun 9, 2014
@parkr
Copy link
Member

parkr commented Jun 9, 2014

And thanks, @penibelst!

@doktorbro doktorbro deleted the remove-coveralls branch June 9, 2014 02:12
@jekyll jekyll locked and limited conversation to collaborators Feb 27, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants