Skip to content
This repository was archived by the owner on Jan 18, 2025. It is now read-only.

Adding passenv to tox coveralls rule. #181

Merged
merged 1 commit into from
May 14, 2015

Conversation

dhermes
Copy link
Contributor

@dhermes dhermes commented May 14, 2015

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling f7bd247 on dhermes:fix-coveralls into * on google:master*.

@nathanielmanistaatgoogle
Copy link
Contributor

LGTM but @craigcitro or @soltanmm will you be a second pair of eyes?

@dhermes
Copy link
Contributor Author

dhermes commented May 14, 2015

If you look in a job, e.g. https://travis-ci.org/google/oauth2client/jobs/62570277 then you see in coveralls it says (line 165)

You have to provide either repo_token in .coveralls.yml, or launch via Travis

craigcitro added a commit that referenced this pull request May 14, 2015
Adding passenv to tox coveralls rule.
@craigcitro craigcitro merged commit e7340f3 into googleapis:master May 14, 2015
@dhermes dhermes deleted the fix-coveralls branch June 24, 2015 05:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants