Skip to content

chore: remove the trailing space that fails the mdformat hook - #4

Merged
JustinKovacich merged 1 commit into
mainfrom
chore/mdformat-changelog
Sep 11, 2026
Merged

chore: remove the trailing space that fails the mdformat hook#4
JustinKovacich merged 1 commit into
mainfrom
chore/mdformat-changelog

Conversation

@JustinKovacich

Copy link
Copy Markdown
Contributor

What

Strips one trailing space from ## Added in CHANGELOG.md.

Why

The mdformat pre-commit hook rewrites that line and exits non-zero when it
modifies a file, so Pre-commit Checks has failed on every pull request since
the initial commit
— including Dependabot's open #3. The nit lives on main,
so every branch inherited a red check it didn't cause.

Verification

Full hook suite run locally against the branch — all 13 pass:

Check for useless excludes...........Passed    Lint GitHub Actions workflow files...Passed
trim trailing whitespace.............Passed    zizmor...............................Passed
fix end of files.....................Passed    typos................................Passed
check yaml...........................Passed    Detect hardcoded secrets.............Passed
check for added large files..........Passed    mdformat.............................Passed
check for merge conflicts............Passed    check json..........................Skipped
mixed line ending....................Passed

Follow-up

With this green, Pre-commit Checks can join the required status checks on the
main ruleset, which currently requires the seven Smoke Test jobs.

🤖 Generated with Claude Code

The mdformat hook rewrites `## Added ` to `## Added` and exits non-zero when
it modifies a file, so Pre-commit Checks has failed on every pull request
since the initial commit — including Dependabot's. The nit is on main, not in
the branches it was failing.

With this fixed, Pre-commit Checks can join the required status checks on the
main ruleset.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@JustinKovacich
JustinKovacich merged commit 0753ea0 into main Sep 11, 2026
18 checks 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.

2 participants