-
Notifications
You must be signed in to change notification settings - Fork 27
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
coveralls + travis + docker #88
Comments
As suggested, you will also need to pass in the If you link to a public repository and build where you are having trouble with I could take a closer look, although I suspect this to be the problem. |
Yes you can add I took as an an example one implementation, here you have:
You can also find |
Apologies for the confusion. I have added the travis provider into the next release version code, but have not released it yet. Until this is released, all settings will need to be done explicitly. The example of how to do this for now unfortunately is to specify all the command-line arguments against the tool, and then also specify all the environment variables used to docker as well. My hope is that soon i can work on a further release including the Travis support automatically, and then you will only need to specify everything in docker once. |
I see the linked PR was a success so am closing this Issue. |
I am using docker container with travis to test and generate coverage report. As I am aware (based on my experience with other languages) when you use coveralls with travis on public repositories you don't need to specify any additional tokens, however with coveralls.net I got this:
I passed the following environment variables to container:
The text was updated successfully, but these errors were encountered: