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 hooks #9510

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add pre-commit hooks #9510

wants to merge 1 commit into from

Conversation

Zopolis4
Copy link
Collaborator

Add pre-commit hooks via https://pre-commit.com/.

Also add py3-pre-commit package.

Tested and working on x86_64.

The changes to .md files and caja.rb are the pre-commit hooks in action.

Run the following to get this pull request's changes locally for testing.

CREW_REPO=https://github.com/Zopolis4/chromebrew.git CREW_BRANCH=clawed crew update

bin/crew Outdated Show resolved Hide resolved
@Zopolis4 Zopolis4 requested a review from satmandu March 20, 2024 20:47
@Zopolis4 Zopolis4 force-pushed the clawed branch 2 times, most recently from e7cde53 to 385cacc Compare March 21, 2024 03:06
@Zopolis4
Copy link
Collaborator Author

As per discussions on slack, I am keeping the code to run rubocop in crew upload, as satmandu uses it in his workflow and it makes contributing easier for new contribors.

.github/PULL_REQUEST_TEMPLATE.md Outdated Show resolved Hide resolved
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.

None yet

3 participants