Skip to content

DO NOT MERGE: Test pre-commit check working as expected#225

Open
sumau wants to merge 1 commit into
mainfrom
sumau-patch-1
Open

DO NOT MERGE: Test pre-commit check working as expected#225
sumau wants to merge 1 commit into
mainfrom
sumau-patch-1

Conversation

@sumau

@sumau sumau commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Test that committing code without running theb pre-commit hook makes the pre-commit hook check github action workflow fail and prints a comment on the PR, but that it's still possible for admins to bypass

@sumau
sumau requested a review from a team as a code owner July 14, 2026 08:18
@github-actions

Copy link
Copy Markdown

Your PR has commits that are missing the Signed-off-by trailer. This is likely due to the pre-commit hook not being configured on your local machine. The usual fix for this issue is to run pre-commit install --install-hooks --overwrite -t commit-msg -t pre-commit, however for more detailed help in setting up the pre-commit hooks, follow the instructions at https://github.com/uktrade/github-standards/blob/main/README.md#usage

@github-actions

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or OpenSSF Scorecard issues found.

Scanned Files

None

@sumau sumau changed the title Update README.md TEST - DO NOT MERGE ME! Jul 14, 2026
@sumau sumau changed the title TEST - DO NOT MERGE ME! TEST PRE COMMIT CHECK - DO NOT MERGE ME! Jul 16, 2026
@sumau sumau changed the title TEST PRE COMMIT CHECK - DO NOT MERGE ME! Test pre-commit check - DO NOT MERGE ME! Jul 16, 2026
@sumau sumau changed the title Test pre-commit check - DO NOT MERGE ME! DO NOT MERGE: Test pre-commit check failing Jul 16, 2026
@sumau sumau changed the title DO NOT MERGE: Test pre-commit check failing DO NOT MERGE: Test pre-commit check working as expected Jul 24, 2026
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.

2 participants