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

Closes #416 - Add code coverage to Coveralls.io #461

Merged
merged 4 commits into from
Mar 16, 2017

Conversation

JanDeDobbeleer
Copy link
Contributor

@JanDeDobbeleer JanDeDobbeleer commented Mar 15, 2017

The API key used in appveyor.yml is still mine. It needs to be replaced before going to master. Create a new integration for posh-git at Coveralls.io and encrypt the project token using the secure variable page on Appveyor.

The build fails because the API key can't be decrypted (it's encrypted with my account). I've setup a build on my account, which can be found here. The result is pictured on Coveralls here.

This closes #416

@JanDeDobbeleer JanDeDobbeleer force-pushed the feature/code_coverage branch 2 times, most recently from 6b0c2e4 to 90d51fd Compare March 15, 2017 11:34
@dahlbyk dahlbyk mentioned this pull request Mar 16, 2017
@dahlbyk
Copy link
Owner

dahlbyk commented Mar 16, 2017

Any idea why my encrypted environment variable isn't available? #462 shows the environment variable is missing altogether, rather than set to empty.

@dahlbyk
Copy link
Owner

dahlbyk commented Mar 16, 2017

As Appveyor's tweeter so helpfully pointed out (and the docs are quite clear), secure variables aren't available in PR builds.

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.

Track Code Coverage (Coveralls.io)
2 participants