Skip to content

Use provided encoding in ensure_text #205

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Merged
merged 1 commit into from
Nov 7, 2019
Merged

Use provided encoding in ensure_text #205

merged 1 commit into from
Nov 7, 2019

Conversation

jakirkham
Copy link
Member

Follow-up to PR ( #201 ), which fixes a typo in ensure_text.

TODO:

  • Unit tests and/or doctests in docstrings
  • tox -e py37 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)

@jakirkham jakirkham merged commit 49c2030 into zarr-developers:master Nov 7, 2019
@jakirkham jakirkham deleted the fix_ensure_text_encoding branch November 7, 2019 18:59
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jakirkham