Skip to content

release-notes: Ignore "NONE" release note - #430

Merged
aanm merged 1 commit into
mainfrom
pr/joe/ignore-k8s-style-empty-release-note
Aug 27, 2026
Merged

release-notes: Ignore "NONE" release note#430
aanm merged 1 commit into
mainfrom
pr/joe/ignore-k8s-style-empty-release-note

Conversation

@joestringer

Copy link
Copy Markdown
Member

We're frequently seeing contributors fail to read instructions and put a
release note containing the text "NONE" in the release note section,
similar to the rules style that Kubernetes expects. Rather than push a
rock up a hill in convincing scores of new contributors to follow the
pattern we expect, let's just ignore any release note that contains the
text "NONE".

We're frequently seeing contributors fail to read instructions and put a
release note containing the text "NONE" in the release note section,
similar to the rules style that Kubernetes expects. Rather than push a
rock up a hill in convincing scores of new contributors to follow the
pattern we expect, let's just ignore any release note that contains the
text "NONE".

Signed-off-by: Joe Stringer <joe@cilium.io>
@joestringer
joestringer requested a review from aanm August 26, 2026 16:06
Comment thread pkg/github/labels_test.go
title: "Ketchup on pizza",
body: "```release-note\r\nNONE\n",
},
want: "Ketchup on pizza",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

No, we don't want ketchup on pizza!

@aanm
aanm merged commit a572606 into main Aug 27, 2026
1 check passed
@aanm
aanm deleted the pr/joe/ignore-k8s-style-empty-release-note branch August 27, 2026 08:16
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