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

Add pre-commit configuration, some new checks and reformat #50

Merged
merged 17 commits into from
Aug 5, 2023

Conversation

Jackenmen
Copy link
Contributor

If you don't want some from the checks that are done here, let me know and I'll remove them.
If you're not okay with doing the reformatting in the same PR that pre-commit configuration is added in, I can also split this PR in two (or in however many pieces you want).

Using pre-commit is as simple as running this in your (preferably global) environment:
pip install -U pre-commit
And then running pre-commit install in the root of your repository.
From now on, pre-commit will automatically run when you make a commit and will let you know if there are issues with any of the checks it runs.

@Jackenmen Jackenmen force-pushed the pre-commit-config branch 3 times, most recently from cc4d940 to 68f44b4 Compare April 20, 2021 11:25
@NeuroAssassin NeuroAssassin merged commit 6ad4ec4 into NeuroAssassin:master Aug 5, 2023
1 check passed
@Jackenmen Jackenmen deleted the pre-commit-config branch August 5, 2023 01:28
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