Modernize the package a bit

pangeo-data / xESMF

Universal Regridder for Geospatial Data

http://xesmf.readthedocs.io/

MIT License

183 stars 32 forks source link

Closed ocefpaf closed 1 year ago

ocefpaf commented 1 year ago

In this PR:

update PyPI GHA
moved as much as I can to pyproject.toml
remove setuptools_scm_git_archive to rely only on the tags to publish new versions
drop py37 from the CIs, this was already dropped by conda-forge
we should not depend on esmpy until we can get a wheel on PyPI xref.: https://github.com/conda-forge/esmpy-feedstock/issues/72

ocefpaf commented 1 year ago

@huard I also want to implement linting with ruff but I'll leave that for another PR.

For now, we still cannot publish on PyPI b/c someone with admin rights to this repo needs to add the PyPI token to the repo secrets.

huard commented 1 year ago

Ok, give me 5 minutes, I'll get the token.

huard commented 1 year ago

Done, there is now a PYPI_TOKEN. Note that we cannot push as is to TestPyPI because I don't have the permissions for xesmf on testpypi.

ocefpaf commented 1 year ago

Note that we cannot push as is to TestPyPI because I don't have the permissions for xesmf on testpypi.

I find testpypi an overkill. Hopefully we test the tarball enough in our CIs to upload directly to PyPI.

PS: if you want to tag a new release after this PR is merged we can test the autopublish.

huard commented 1 year ago

Thanks !