dtrx-py / dtrx

Do The Right Extraction
GNU General Public License v3.0
230 stars 10 forks source link

🔨 Update to use `build` for package building #22

Closed noahp closed 2 years ago

noahp commented 2 years ago

Still using setuptools as the build backend, but switch to https://github.com/pypa/build for the frontend (it builds both sdist and wheel in the same command, and is PEP 517).

Several related updates:

The setup.cfg file was formatted with the quite nice setup-cfg-fmt package (https://github.com/asottile/setup-cfg-fmt).

This also removes twine from the setup-requires dependencies. Fixes #19.

And moving the package configuration to setup.cfg, I noticed the subprocess32 backport dependency was not specified correctly when building the wheel (it was set based on the build environment, not the install environment), fixed now.

noahp commented 2 years ago

@ChrisJefferson Let me know what you think- this is a somewhat disruptive rework of the packaging scheme (still using setuptools though).

ChrisJefferson commented 2 years ago

Thanks, this looks really good.