Skip to content

JANITORIAL: add pre-commit configuration file#7000

Merged
sev- merged 1 commit intoscummvm:masterfrom
whoozle:add-pre-commit
Nov 29, 2025
Merged

JANITORIAL: add pre-commit configuration file#7000
sev- merged 1 commit intoscummvm:masterfrom
whoozle:add-pre-commit

Conversation

@whoozle
Copy link
Copy Markdown
Contributor

@whoozle whoozle commented Nov 1, 2025

Add pre-commit configuration. Pre-commit is now de-facto industry-standard of running linters/formatters for git.

With provided config, it would run clang-format on your files to maintain consistent code formatting.
If installed as pre-commit hook it would also check that your commit message has prefix.

This is not mandatory, you can opt-in by installing pre-commit, pip install --upgrade pre-commit, then run pre-commit install in scummvm directory.

To opt-out run pre-commit uninstall

@whoozle whoozle changed the title Add pre-commit configuration file JANITORIAL: add pre-commit configuration file Nov 1, 2025
@sev-
Copy link
Copy Markdown
Member

sev- commented Nov 2, 2025

Thank you for looking into that

You can opt-in by install pre-commit, `pip install --upgrade pre-commit`,
then run `pre-commit install` in scummvm directory.
@sev-
Copy link
Copy Markdown
Member

sev- commented Nov 29, 2025

Thank you

@sev- sev- merged commit 5aaf476 into scummvm:master Nov 29, 2025
1 check passed
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.

3 participants