chore(website): remove codename field from release definitions and template#25554
chore(website): remove codename field from release definitions and template#25554thomasqueirozb wants to merge 2 commits into
Conversation
thomasqueirozb
added
the
no-changelog
github-actions
Bot
added
docs review on hold
github-actions
Bot
added
the
docs review on hold
github-actions
Bot
removed
the
docs review on hold
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: dbb83d72fd
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| </p> | ||
| {{ end }} | ||
|
|
||
| {{ with $highlights }} |
There was a problem hiding this comment.
This makes the change user-visible for historical releases, not just a schema cleanup: the removed block was the only renderer for $release.codename, and several entries deleted in this commit had real values such as 0.10.0's Lamport Clock, 0.12.0's Vector Remap Language, and 0.4.0's Hello World Again. Those release pages will now permanently lose that published metadata, so keep rendering/preserving non-empty codenames if the goal is only to remove unused empty fields.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Let's leave the codenames (as comments, not part of the schema) next to version number.
3 participants
Summary
Removes the unused
codenamefield from all release CUE definitions, the release schema, the CUE generator, and the release page HTML template.Vector configuration
NA
How did you test this PR?
NA
Change Type
Is this a breaking change?
Does this PR include user facing changes?
no-changeloglabel to this PR.References
NA