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

Makefile to tox #24

Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Open

Makefile to tox #24

wants to merge 13 commits into from

Conversation

cylussec
Copy link

@cylussec cylussec commented Mar 3, 2023

Tox is more Pythonic than Makefiles. This PR has the tox files, switches from setup.py/setup.cfg/requirements.txt to mainly using setup.cfg (which is the current standard), moves some of your black/flake8/isort checks into precommit hooks (as well as being run in Github Actions).

I can't take credit for all of this. Whenever I start a new project, I use Pyscaffold to create a lot of the base files, and a lot of these are modified files that come with that.

@cylussec
Copy link
Author

cylussec commented Mar 3, 2023

Ugh... I just realized that I was branching off the same branches, so this last PR has all of the changes. If you want some of these changes and not all of them, I am happy to do cleaner PRs for each of these from main.

I did all of these changes together because they are a little dependent on each other, so thats my bad.

@tetracionist
Copy link
Contributor

Hey @cylussec,

Starting to have a look through your PRs for this.
Will add this to the main repo!

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