Skip to content
This repository has been archived by the owner on Dec 10, 2018. It is now read-only.

only do coveralls when possible #23

Merged
merged 1 commit into from
Nov 5, 2014
Merged

Conversation

jonlawlor
Copy link
Contributor

Travis only gives secure environment variables when the origin and
target repo are the same in a pull request. We shouldn’t break the CI
when coverage isn’t available.

Travis only gives secure environment variables when the origin and
target repo are the same in a pull request.  We shouldn’t break the CI
when coverage isn’t available.
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 142550c on only-coveralls-if-branch into 6c636e8 on master.

@jonlawlor
Copy link
Contributor Author

Small change to travis.yml to avoid doing coverage when the environment variables are not available.

Follows up the issues encountered in #22

@jonlawlor
Copy link
Contributor Author

I'd like to make similar changes to the other .travis.yml files in the other gonum repos. If you have any questions, then let's discuss it here instead of across 8 different pull requests.

jonlawlor added a commit that referenced this pull request Nov 5, 2014
@jonlawlor jonlawlor merged commit 16818d9 into master Nov 5, 2014
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.

2 participants