Skip to content

Remove Python 2 code#368

Merged
jakirkham merged 1 commit intozarr-developers:mainfrom
DimitriPapadopoulos:python3
Oct 28, 2022
Merged

Remove Python 2 code#368
jakirkham merged 1 commit intozarr-developers:mainfrom
DimitriPapadopoulos:python3

Conversation

@DimitriPapadopoulos
Copy link
Copy Markdown
Contributor

@DimitriPapadopoulos DimitriPapadopoulos commented Oct 27, 2022 •
edited
Loading

TODO:

  • Unit tests and/or doctests in docstrings
  • tox -e py310 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
  • GitHub Actions CI passes
  • Test coverage to 100% (Coveralls passes)

@DimitriPapadopoulos DimitriPapadopoulos changed the title Remove Python2 code Remove Python 2 code Oct 27, 2022
@jakirkham
Copy link
Copy Markdown
Member

jakirkham commented Oct 28, 2022

Thanks Dimitri! 🙏

Could you please merge/rebase and an add a release entry?

jakirkham
jakirkham reviewed Oct 28, 2022
View reviewed changes
@DimitriPapadopoulos
Remove Python 2 code
a7b458b 
 Veuillez saisir le message de validation pour vos modifications. Les lignes
@jakirkham jakirkham enabled auto-merge (squash) October 28, 2022 16:55
@jakirkham jakirkham merged commit 0a2ec9f into zarr-developers:main Oct 28, 2022
@DimitriPapadopoulos DimitriPapadopoulos deleted the python3 branch October 28, 2022 18:08
@jakirkham jakirkham mentioned this pull request Oct 28, 2022
Merged
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jakirkham jakirkham jakirkham approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@DimitriPapadopoulos @jakirkham