build: remove setup.py and other build changes #79
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This PR provides changes to gradually harmonize some of the build process. Less moving pieces, and closer to best practices.
Changes
pip install --no-deps .)requirements-test.txtintopyproject.toml, using pip "dependency group" to bake this intopyproject.tomland install only the test deps without the main deps.pre-commit-config.yamlbecause it's personal for nowRelated Issues
The 'main' deps declared in
pyproject.tomlwill break the production environment. I've confirmed that setup.py route only worked because it skipped all the deps declared in pyproject.toml :)Testing
Locally all dandy, launching a test job now.
Other Notes