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

just to get coveralls details #509

Closed
wants to merge 3 commits into from
Closed

just to get coveralls details #509

wants to merge 3 commits into from

Conversation

coolo
Copy link
Contributor

@coolo coolo commented Jan 18, 2016

No description provided.

@coolo
Copy link
Contributor Author

coolo commented Jan 20, 2016

the coverage dropped by 14% and coveralls didn't care to comment. You have to check the coveralls.io page to see it - which might be fine if you have a suspicion

@coolo coolo closed this Jan 20, 2016
@coolo coolo deleted the test_coveralls branch January 20, 2016 09:24
@okurz
Copy link
Member

okurz commented Jan 20, 2016

I don't have project permissions to check this in detail but could you try to disable and re-enable the sync within coveralls? Maybe coveralls is missing the github API access token to post back the result

@coolo
Copy link
Contributor Author

coolo commented Jan 20, 2016

I did that on monday actually

@coolo
Copy link
Contributor Author

coolo commented Jan 20, 2016

and coveralls is posting under its own account anyway, so there shouldn't be API access required to post. The token is only required to query my personal projects

@okurz
Copy link
Member

okurz commented Jan 20, 2016 via email

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.

2 participants