Attempt to fix coveralls

[mzjp2]

Let's see if this works...

TODO:

• Add unit tests and/or doctests in docstrings
• Add docstrings and API docs for any new/modified user-facing classes and functions
• New/modified features documented in docs/tutorial.rst
• Changes documented in docs/release.rst
• AppVeyor and Travis CI passes
• Test coverage is 100% (Coveralls passes)

[jakirkham]

Are you able to see this page?

[mzjp2]

From the Coveralls docs on Travis Pro, it says to do this..

First, if you are using Travis CI , make sure it's building correctly. If you're using Travis Pro, add the following lines to a .coveralls.yml file in your repository's root:

service_name: travis-pro
repo_token: f9ihk8Pur6zETkTohfPmOPwbEWrZTkKK8

That's the generic advice when not using a coveralls client like we are (hence the use of the .coveralls.yml file. We're using https://github.com/coveralls-clients/coveralls-python and they had this tip in their docs: https://coveralls-python.readthedocs.io/en/latest/usage/configuration.html

[jakirkham]

FWIW I regenerated the token already 😉

It seems they mention COVERALLS_REPO_TOKEN should be set. We might need travis encrypt to obscure this value.

[mzjp2]

It seems to say that it'll find the token automatically: 🤷‍♂ how it does that, I don't know

If coveralls-python is being run on TravisCI, it will automatically set the token for communication with coveralls.io. Otherwise, you should set the environment variable COVERALLS_REPO_TOKEN, which can be found on the dashboard for your project in coveralls.io:

[mzjp2]

Aha, running coveralls debug in travis yields:

coverage.misc.CoverageException: Couldn't use data file '/home/travis/build/zarr-developers/zarr-python/.coverage': Looks like a coverage 4.x data file. Are you mixing versions of coverage?

[jakirkham]

Good catch! Maybe we can update these lines?

[mzjp2]

So they've all been reported to coveralls succesfully at least, but we're not getting the Github Status update. I might just merge this to master anyway and see if that helps any

[jakirkham]

Happy with merging to master. Perhaps ( lemurheavy/coveralls-public#1408 ) is just indicating a service outage.

[mzjp2]

it works! :)

[jakirkham]

Thanks for working on this @mzjp2! 😄

[jakirkham]

Bumping coverage a bit with PR ( #542 ).

[jakirkham]

If you have a chance @mzjp2, would you be able to look at the numcodecs repo as well? Guessing it needs similar changes.

[mzjp2]

Done @jakirkham, should be all good now, good spot. :)

[jakirkham]

Wow! That was fast. Thanks Zain! 😄