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.

Sign up for GitHub

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

Jump to bottom

Add more pre-commit hooks #24

Merged
merged 2 commits into from
Jul 15, 2023
Merged

Add more pre-commit hooks #24

merged 2 commits into from
Jul 15, 2023

Conversation

diazona
Copy link
Owner

@diazona diazona commented Jul 12, 2023

Pretty simple, this PR adds some basic hooks from https://github.com/pre-commit/pre-commit-hooks to do things like checking whitespace and syntax.

diazona added 2 commits July 11, 2023 23:09
@diazona
Add some more pre-commit hooks
298eb80 
This commit adds several pre-commit hooks from pre-commit's own default
hook repository. The new hooks will do things like checking for extra
whitespace and catching syntax errors.
@diazona
Remove a trailing newline caught by pre-commit
b11bb12
@diazona diazona added this to the Initial release milestone Jul 12, 2023
@diazona diazona requested a review from sjlongland July 12, 2023 06:23
@diazona diazona assigned diazona and sjlongland and unassigned diazona Jul 12, 2023
sjlongland
sjlongland approved these changes Jul 15, 2023
View reviewed changes
Copy link
Collaborator

@sjlongland sjlongland left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Definitely learning some validation tricks with this project. :-)

@sjlongland sjlongland merged commit b8ff80c into main Jul 15, 2023
@sjlongland sjlongland deleted the pre-commit-hooks/2/dev branch July 15, 2023 00:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sjlongland sjlongland sjlongland approved these changes

Assignees

@sjlongland sjlongland

Labels
None yet
Projects
None yet
Milestone
Initial release
Development

Successfully merging this pull request may close these issues.
2 participants
@diazona @sjlongland