feat: add pre-commit configuration for linting and commit message enforcement (#117) #202

shoko · 2026-04-07T05:51:55+02:00

shoko commented

2026-04-07 05:51:55 +02:00

No description provided.

shoko added 1 commit 2026-04-07 05:51:55 +02:00

feat: add pre-commit configuration for linting and commit message enforcement (#117 ) e86a309059

han commented

2026-04-07 06:03:05 +02:00

First-time contributor

is it possible to add pre commit hook so anyone who want to commit in this repo will have their pre commit hook to run and fix it for them. what do you think?

shoko commented

2026-04-07 06:33:07 +02:00

This PR adds the pre-commit configuration! Once merged, contributors can run pre-commit install to enable the hooks locally. The config includes:

shellcheck - Lints bash scripts
ruff - Lints and formats Python
commitizen - Enforces Conventional Commits format on commit messages

After merging, you would need to:

Run pre-commit install to install the hooks
On each git commit, the hooks will run automatically

Note: The hooks run locally on the developer's machine - they don't auto-fix on push, but they will catch issues before commit.

This PR adds the pre-commit configuration! Once merged, contributors can run `pre-commit install` to enable the hooks locally. The config includes: - **shellcheck** - Lints bash scripts - **ruff** - Lints and formats Python - **commitizen** - Enforces Conventional Commits format on commit messages After merging, you would need to: 1. Run `pre-commit install` to install the hooks 2. On each `git commit`, the hooks will run automatically Note: The hooks run locally on the developer's machine - they don't auto-fix on push, but they will catch issues before commit.

han commented

2026-04-07 06:34:27 +02:00

First-time contributor

can you write about it in CONTRIBUTING.md so anyone who contribute will know to run pre-commit install?

shoko added 1 commit 2026-04-07 06:35:18 +02:00

docs: add pre-commit hooks section to CONTRIBUTING.md (#117 ) 930d0e53b5

shoko commented

2026-04-07 06:35:22 +02:00

Done! Added a Pre-commit Hooks section to CONTRIBUTING.md with setup instructions and commit message format. The PR now includes both the hook configuration and documentation.

han commented

2026-04-07 06:36:31 +02:00

First-time contributor

lgtm

shoko merged commit efcec4e122 into main

2026-04-07 08:02:57 +02:00

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: shoko/kugetsu#202