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

Use correct label in PHP Backport documentation #65908

Merged
merged 1 commit into from
Oct 8, 2024

Conversation

getdave
Copy link
Contributor

@getdave getdave commented Oct 7, 2024

What?

Uses correct label in the PHP Backporting documentation

Why?

Because details matter...

How?

Corrects label from Backport to WP Core to Backported to WP Core (past tense).

Testing Instructions

Read words.

Testing Instructions for Keyboard

Screenshots or screencast

Screen Shot 2024-10-07 at 11 06 10

@getdave getdave added the [Type] Developer Documentation Documentation for developers label Oct 7, 2024
@getdave getdave self-assigned this Oct 7, 2024
Copy link

github-actions bot commented Oct 7, 2024

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: getdave <[email protected]>
Co-authored-by: kevin940726 <[email protected]>

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

Copy link
Member

@kevin940726 kevin940726 left a comment

Choose a reason for hiding this comment

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

Thanks!

@@ -31,7 +31,7 @@ There are however certain exceptions to that rule. PRs with the following criter

- Does not contain changes to PHP code.
- Has label `Backport from WordPress Core` - this code is already in WP Core and is being synchronized back to Gutenberg.
- Has label `Backport to WordPress Core` - this code has already been synchronized to WP Core.
- Has label `Backported to WordPress Core` - this code has already been synchronized to WP Core.
Copy link
Member

Choose a reason for hiding this comment

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

Should we also change this to

Suggested change
- Has label `Backported to WordPress Core` - this code has already been synchronized to WP Core.
- Has label `Backported to WP Core` - this code has already been synchronized to WP Core.

to match the actual label name?

@getdave getdave force-pushed the fix/label-typo-in-php-backport-docs branch from 0cd6375 to 8dce5ce Compare October 8, 2024 12:36
@getdave getdave enabled auto-merge (squash) October 8, 2024 12:37
@getdave getdave merged commit 076be0c into trunk Oct 8, 2024
62 checks passed
@getdave getdave deleted the fix/label-typo-in-php-backport-docs branch October 8, 2024 13:13
@github-actions github-actions bot added this to the Gutenberg 19.5 milestone Oct 8, 2024
karthick-murugan pushed a commit to karthick-murugan/gutenberg that referenced this pull request Nov 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Developer Documentation Documentation for developers
Projects
Development

Successfully merging this pull request may close these issues.

2 participants