Skip to content

Fix build#244

Merged
joshmoore merged 5 commits into
zarr-developers:masterfrom
Czaki:fix_build
Sep 10, 2020
Merged

Fix build#244
joshmoore merged 5 commits into
zarr-developers:masterfrom
Czaki:fix_build

Conversation

@Czaki

@Czaki Czaki commented Sep 10, 2020

Copy link
Copy Markdown
Contributor

[Description of PR]
fix #242
describen here #242 (comment)
TODO:

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

@Czaki

Czaki commented Sep 10, 2020

Copy link
Copy Markdown
Contributor Author

It looks like it works.
I'm not sure if keep test_ dist_pypi

EDIT
still problem with sdist

@Czaki Czaki marked this pull request as ready for review September 10, 2020 12:02
@Czaki

Czaki commented Sep 10, 2020

Copy link
Copy Markdown
Contributor Author

@joshmoore ready

@joshmoore joshmoore left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @Czaki, for tackling this. Very much appreciated. A few questions though.

run: python -m pip list

- name: Run tests
run: pytest -v --pyargs numcodecs No newline at end of file

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the only location currently running pytest on OS X, no?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My bad. Only remark is that Github actions limit macos concurrent job to 5.

And wheel are tested using pytest, so it is not only one place.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gotcha. GitHub Action search only works on the logs that you've opened. Happy to have ci-osx.yml re-removed, but I'd propose let's not hold up this since others like @martindurant are having issues due to my 0.7.0 release.

- name: test
run: |
ls
ls dist

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you mean to try installing the wheel here? Or was this just temporary debugging?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

debug. It allow to check if all files are properly placed.

@joshmoore joshmoore merged commit 77230da into zarr-developers:master Sep 10, 2020
@jakirkham

Copy link
Copy Markdown
Member

Awesome! Thanks so much Grzegorz! Also thanks Josh for reviewing and trying another release (looks like it is working 🎉). 😄

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.

Numcodecs 0.7.0 fails to install, missing blosc.c

3 participants