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 pre commit hook #12

Merged
merged 3 commits into from Jul 13, 2021
Merged

Add pre commit hook #12

merged 3 commits into from Jul 13, 2021

Conversation

Casyfill
Copy link
Contributor

@Casyfill Casyfill commented Jul 11, 2021 •
edited

Adds pre-commit hooks, which can be referred to and ran on commit/push/merge from any repository. I just tested this with my fork of catalyst and it works without any hiccup "on my machine" ( link )

PR to catalyst main repo with use of those hooks is here

All docs on pre-commit are here

@Casyfill Casyfill mentioned this pull request Jul 11, 2021
Merged
20 tasks
@Scitator Scitator requested review from Scitator and bagxi July 13, 2021 16:31
@Scitator Scitator merged commit 965435e into catalyst-team:master Jul 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ditwoo ditwoo ditwoo approved these changes

@Scitator Scitator Scitator approved these changes

@bagxi bagxi Awaiting requested review from bagxi

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@Casyfill @ditwoo @Scitator