-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Automated Changelog: Rename "Editor" grouping to "Post Editor" to avoid ambiguity with other editors #34093
Conversation
Size Change: +12 B (0%) Total Size: 1.03 MB
ℹ️ View Unchanged
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks so much for addressing all of that feedback, you were all so quick! |
Thanks, @gziolo ! I was considering removing entirely the |
They are more granular than the section name so I could live with them, but if you think it makes sense to remove them then we can surely do it as a follow-up. The most important fact is that the changelog is looking very close to the final version used when publishing the post ❤️ |
Description
Finishes and closes #33719.
This PR renames the
Editor
feature grouping toPost Editor
to differentiate it from other Editors.How has this been tested?
The snapshot in the unit test got updated to reflect the latest changes.
Run
npm run changelog -- --milestone="Gutenberg 11.3"
and you should see the same changelog generated.Checklist:
*.native.js
files for terms that need renaming or removal).