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: Improve dependencies handling #223

Merged
merged 3 commits into from
Apr 2, 2019

Conversation

nbraud
Copy link
Contributor

@nbraud nbraud commented Apr 1, 2019

@nbraud
Copy link
Contributor Author

nbraud commented Apr 1, 2019

I had to create a setup.cfg to define the necessary alias for pytest-runner to be called by setup.py test, but I didn't try to convert more of setup.py to setup.cfg, as that felt out-of-scope.

I can do so in a follow-up PR though :)

@nbraud
Copy link
Contributor Author

nbraud commented Apr 1, 2019

PS: The file is called requirements-tests.txt rather than test-requirements.txt because there's already a requirements-docs.txt; I asked in #216 whether the convention should be changed, but got no answer.

@nbraud
Copy link
Contributor Author

nbraud commented Apr 1, 2019

BTW, once this is merged, #216 and #220 should be updated.
Update: I rebased those 2 PRs to include the first 2 commits of this branch, so this can all go without blocking on me.

@pathunstrom pathunstrom merged commit 1cde076 into ppb:master Apr 2, 2019
@nbraud nbraud deleted the setup.py/dependencies 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