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

Fix get codec #79

Merged
merged 2 commits into from
May 13, 2018
Merged

Conversation

jeromekelleher
Copy link
Member

Simple fix for #78.

Also removed a minor spam output from running tests.

TODO:

  • Unit tests and/or doctests in docstrings
  • tox -e py36 passes locally
  • tox -e py27 passes locally
  • Docstrings and API docs for any new/modified user-facing classes and functions
  • Changes documented in docs/release.rst
  • tox -e docs passes locally
  • AppVeyor and Travis CI passes
  • Test coverage to 100% (Coveralls passes)

@alimanfoo
Copy link
Member

Thanks @jeromekelleher. I've some other stuff to get done today but will follow this and #77 up first thing next week.

@alimanfoo alimanfoo added this to the 0.6.0 milestone May 13, 2018
@alimanfoo alimanfoo merged commit 6ecedcd into zarr-developers:master May 13, 2018
@alimanfoo
Copy link
Member

Looks good, thank you.

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.

2 participants