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

setup.py: Move all configuration to setup.cfg #224

Merged
merged 8 commits into from
Apr 2, 2019

Conversation

nbraud
Copy link
Contributor

@nbraud nbraud commented Apr 1, 2019

Follow-up on #223

  • Move all setup.py configuration and metadata to setup.cfg, except for dependencies.
  • Add missing metadata (classifiers).
  • Add PEP 518 metadata (pyproject.toml) defining which build system is used.
  • Make setup.py executable.

@nbraud nbraud force-pushed the setup.py/setup.cfg branch from 80d72c1 to 09998e9 Compare April 1, 2019 22:16
Copy link
Collaborator

@pathunstrom pathunstrom left a comment

Choose a reason for hiding this comment

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

Been wanting to make a switch to setup.cfg, so very happy with all of this work, thanks a lot!

@nbraud
Copy link
Contributor Author

nbraud commented Apr 1, 2019

You are welcome :)

@pathunstrom pathunstrom merged commit 5c230df into ppb:master Apr 2, 2019
@nbraud nbraud deleted the setup.py/setup.cfg branch April 2, 2019 10:16
@nbraud nbraud mentioned this pull request Apr 2, 2019
5 tasks
bors bot added a commit to ppb/ppb-vector that referenced this pull request Apr 4, 2019
141: setup.py improvements r=astronouth7303 a=nbraud

Mostly the same thing as ppb/pursuedpybear#223 and ppb/pursuedpybear#224 :

- [x] Rename `*-requirements.txt` to `requirements-*.txt` to follow PPB's convention.
- [x] Extract runtime dependencies to `requirements.txt`.
- [x] Move all metadata from `setup.py` to `setup.cfg`.
- [x] Declare test dependencies in `setup.py`, integrate with `pytest`.
- [x] Use `pyproject.toml` to explicitly declare that we are using `setuptools`.

Co-authored-by: Nicolas Braud-Santoni <nicolas@braud-santoni.eu>
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.

2 participants