rpless / cover-coveralls Goto Github PK

I think we need to capture current-error-port as well as the current-output-port

It looks like #11 wasn't quite enough to fix the version issue since the version of this package on the catalog points to the release branch. Would you be willing to merge master into that branch to fix the published version?

Thanks!

With the current example Travis script, submitting test coverage counts as part of the testing commands. This means if the tests pass but coverage submission fails (for example due to network issues or Coveralls setup issues), the build fails with a test failure state.

I think most would prefer if the coverage process was independent of the testing process, but submitted only when tests pass. This can be done in travis by moving the coverage command from the script: block to an after_success: block (see the Travis Build Lifecycle docs for details):

script:
 - raco test $TRAVIS_BUILD_DIR # run tests. you wrote tests, right?

after_success:
 - raco cover -f coveralls -d $TRAVIS_BUILD_DIR/coverage . # generate coverage information for coveralls

Some people want coverage submission failing to fail the build, but I think most don't and thus this makes for a better default.

see: https://travis-ci.org/LeifAndersen/racket-video-backup/jobs/278394450

I've been having trouble getting started & am currently stuck. Here's my full experience:

1. Downloaded cover via raco. Copied the .travis.yml from this repo into my repo.
2. Enabled my repo on Travis & Coveralls
3. Travis-CI build passes. No coverage report generated.
4. Try raco cover -b -f coveralls . locally. Fails with cover: given unknown coverage output format: "coveralls"
5. Install cover-coveralls through raco. Try raco cover -b -f coveralls ., fails with No repo token or ci service detected.
6. Run export COVERALLS_REPO_TOKEN=MY...KEY. Run raco cover -b -f coveralls .. Fails with racket: exec failed (Permission denied; errno=13) coveralls: request to coveralls failed

What's the preferred way to set my COVERALLS_REPO_TOKEN?

Transferring this issue from florence/cover#17.

Coveralls seems to have change their API. They now want a "source_digest" (MD5 hash of the file) instead of the file itself.

E.g. racket-collections

This seems to be because of this change from earlier this year. See here for a possible explanation.

Probably the right fix is upstream on cover-lib, but creating an issue here for additional context and possibly independent resolution.

Transferred from florence/cover#57.