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

Migrate from setup.py to pyproject.toml #297

Merged
merged 1 commit into from
Mar 5, 2023

Conversation

slonopotamus
Copy link
Collaborator

@slonopotamus slonopotamus commented Mar 4, 2023

pyproject.toml is a mainstream (as of today) way to organize Python packages.

There are multiple reasons for that, I'll just leave some links:

I've also took liberty to add myself to authors list in metadata, hopefully 20% of commits justifies this.

Unfortunately, this stuff doesn't properly work with default pip shipped with Python 3.6, so it has to be dropped. Officially, Python 3.6 became EOL in Dec, 2021.


Implementation details:

  1. Version is now specified in pyproject.toml. version.py loads that using Python metadata machinery
  2. All code moved to src directory in order to stop setuptools from choking on our resources dir. See https://setuptools.pypa.io/en/latest/userguide/package_discovery.html for details.

`pyproject.toml` is a mainstream (as of today) way to organize Python packages.

There are multiple reasons for that, I'll just leave some links:

* https://peps.python.org/pep-0517/
* https://peps.python.org/pep-0518/
* https://peps.python.org/pep-0621/
* https://blog.ganssle.io/articles/2021/10/setup-py-deprecated.html

I've also took liberty to add myself to authors list in metadata, hopefully 21% of commits justifies this.

Unfortunately, this stuff doesn't properly work with default pip/setuptools shipped with Python 3.6, so it has to be dropped.
@slonopotamus slonopotamus merged commit f668317 into adamrehn:master Mar 5, 2023
@slonopotamus slonopotamus deleted the pyproject.toml branch March 5, 2023 12:53
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.

1 participant