Skip to content

Docs: Use gutenberg instead of Gutenberg in package name - #71790

Merged
t-hamano merged 2 commits into
trunkfrom
fix/capitalization-in-package-name
Sep 30, 2025
Merged

t-hamano merged 2 commits into
trunkfrom
fix/capitalization-in-package-name

Conversation

@shail-mehta

Copy link
Copy Markdown
Member

What?

Used gutenberg instead of Gutenberg in package tags for consistency.

Why? How?

  • Used gutenberg instead of Gutenberg in package tags for consistency in below files.
  1. lib/class-wp-theme-json-resolver-gutenberg.php
  2. lib/class-wp-theme-json-schema-gutenberg.php
  3. lib/experimental/posts/load.php

Testing Instructions

  1. Open mentioned files
  2. See package name.

@shail-mehta shail-mehta self-assigned this Sep 21, 2025
@shail-mehta shail-mehta added the [Type] Developer Documentation Documentation for developers label Sep 21, 2025
@shail-mehta
shail-mehta marked this pull request as ready for review September 21, 2025 04:21
@github-actions

github-actions Bot commented Sep 21, 2025 •

Copy link
Copy Markdown

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: shail-mehta <shailu25@git.wordpress.org>
Co-authored-by: t-hamano <wildworks@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@shail-mehta shail-mehta added the No Core Sync Required Indicates that any changes do not need to be synced to WordPress Core label Sep 21, 2025

@t-hamano t-hamano left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the PR!

Is there a reason why you only updated files in the lib directory? I think there are many other places you could update besides the lib directory.

@shail-mehta

Copy link
Copy Markdown
Member Author

Apologies, I hadn’t noticed it was also in a different folder. I’ll review and update the PR accordingly.

@shail-mehta

Copy link
Copy Markdown
Member Author

I have updated PR accordingly. Once you get a chance could you please review it?

@t-hamano t-hamano left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM 👍

@t-hamano
t-hamano merged commit 4932e0c into trunk Sep 30, 2025
70 checks passed
@t-hamano
t-hamano deleted the fix/capitalization-in-package-name branch September 30, 2025 00:47
@github-actions github-actions Bot added this to the Gutenberg 21.8 milestone Sep 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

No Core Sync Required Indicates that any changes do not need to be synced to WordPress Core [Type] Developer Documentation Documentation for developers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants