Skip to content

docs: mention the skip-changelog label#5366

Merged
jamescrosswell merged 1 commit into
mainfrom
docs/mention-skip-changelog-label
Jul 10, 2026
Merged

docs: mention the skip-changelog label#5366
jamescrosswell merged 1 commit into
mainfrom
docs/mention-skip-changelog-label

Conversation

@Flash0ver

Copy link
Copy Markdown
Member

Summary

Mention the skip-changelog GitHub label in regards to the auto-changelog generated by Craft.

Remarks

When reviewing #5178, an Agent mentioned:

...
Remaining minor/low-severity items (not blocking)

1. Missing #skip-changelog — per this repo's convention (CLAUDE.md), non-user-facing changes should carry this tag. This PR touches only dev tooling/docs, never ships in the NuGet packages, and the title (Add optional pre-commit hook...) doesn't follow the type: description convention used elsewhere in this repo's changelog (chore:, docs:, etc.). Actionable: add #skip-changelog to the description, or retitle to chore: add optional pre-commit hook for code formatting.

...

The Agent is currently oblivious to the skip-changelog GitHub label:
image

This change should be teaching the Agent to consider the Labels too.

#skip-changelog

@Flash0ver Flash0ver self-assigned this Jul 9, 2026
@Flash0ver Flash0ver added the skip-changelog Suppress automatic changelog generation via Craft label Jul 9, 2026
@Flash0ver Flash0ver marked this pull request as ready for review July 9, 2026 18:35
@Flash0ver Flash0ver requested a review from jamescrosswell as a code owner July 9, 2026 18:35
@github-actions github-actions Bot added the risk: low PR risk score: low label Jul 9, 2026
@jamescrosswell jamescrosswell merged commit cf22849 into main Jul 10, 2026
22 checks passed
@jamescrosswell jamescrosswell deleted the docs/mention-skip-changelog-label branch July 10, 2026 00:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk: low PR risk score: low skip-changelog Suppress automatic changelog generation via Craft

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants