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

test: replace pytest-flake8 with pre-commit #91

Merged
merged 2 commits into from
Dec 21, 2021
Merged

Conversation

miketheman
Copy link
Owner

@miketheman miketheman commented Dec 21, 2021

There's weird behaviors with Flake8 interacting with pytest.

Instead of using pytest to run the tests, make it a pre-commit hook.

Resolves #82

After merging this, sign up with pre-commit.ci to have it run automatically on push.

Signed-off-by: Mike Fiedler miketheman@gmail.com

There's weird behaviors with Flake8 interacting with pytest.

Instead of using pytest to run the tests, make it a pre-commit hook.

Resolves #82

Signed-off-by: Mike Fiedler <miketheman@gmail.com>
@miketheman miketheman added the chore General maintenance items label Dec 21, 2021
@miketheman miketheman self-assigned this Dec 21, 2021
Signed-off-by: Mike Fiedler <miketheman@gmail.com>
@codeclimate
Copy link

codeclimate bot commented Dec 21, 2021

Code Climate has analyzed commit c54ebb4 and detected 0 issues on this pull request.

View more on Code Climate.

@miketheman miketheman merged commit 7f95ca9 into main Dec 21, 2021
@miketheman miketheman deleted the pre_commit_82 branch December 21, 2021 21:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore General maintenance items
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Replace pytest-flake8 with a pre-commit hook
1 participant