Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Do not require changelog entry for docs #278

Merged
merged 2 commits into from
Feb 6, 2025

Conversation

ptgott
Copy link
Contributor

@ptgott ptgott commented Oct 9, 2024

Docs-only changes do not require a changelog entry, as we reserve the
changelog for changes to the product. Edit the changelog workflow to
ignore docs-only changes.

@ptgott ptgott requested a review from a team as a code owner October 9, 2024 17:32
@ptgott ptgott marked this pull request as draft October 9, 2024 17:33
@ptgott

This comment was marked as resolved.

@zmb3

This comment was marked as resolved.

@ptgott ptgott force-pushed the paul.gottschling/2024-07-docs-no-changelog branch 2 times, most recently from ca59a6c to 3e7173c Compare October 11, 2024 20:42
@ptgott ptgott changed the title Bot: Label docs-only changes no-changelog Do not require changelog entry for docs Oct 11, 2024
@ptgott ptgott marked this pull request as ready for review October 11, 2024 20:43
@ptgott ptgott force-pushed the paul.gottschling/2024-07-docs-no-changelog branch from 3e7173c to 7f2d783 Compare January 17, 2025 20:08
@ptgott ptgott force-pushed the paul.gottschling/2024-07-docs-no-changelog branch from 7f2d783 to 9e22a95 Compare January 31, 2025 20:35
Copy link
Contributor

@fheinecke fheinecke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This needs changes to maintain compatibility with release tooling, as discussed on slack here: https://gravitational.slack.com/archives/C0351AUPJ8K/p1738617752346529

Docs-only changes do not require a changelog entry, as we reserve the
changelog for changes to the product. Edit the changelog workflow to
ignore docs-only changes.
@ptgott ptgott force-pushed the paul.gottschling/2024-07-docs-no-changelog branch from 9e22a95 to 5f783ba Compare February 6, 2025 18:21
Copy link
Contributor

@fheinecke fheinecke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM outside of magic string

bot/internal/bot/label.go Outdated Show resolved Hide resolved
bot/internal/bot/label_test.go Outdated Show resolved Hide resolved
Automatically add the `no-changelog` label for docs-only changes. This
reduces friction for docs authors, since docs-only changes usually don't
require changelog entries. Do this instead of automatically skipping the
changelog check for docs-only changes, since our release workflow often
requires us to add or check the `no-changelog` label manually.
@ptgott ptgott force-pushed the paul.gottschling/2024-07-docs-no-changelog branch from 5f783ba to 9078af9 Compare February 6, 2025 18:31
@fheinecke fheinecke merged commit ab80556 into main Feb 6, 2025
8 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.

3 participants