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

Fixes typo in class PHPDoc comment #47261

Merged

Conversation

welenofsky
Copy link
Contributor

What?

In the process of looking into the new webfonts functionality I noticed a typo in the PHPDoc comment for the class WP_Webfonts_Provider

Why?

This PR would update the method in the comment from "set_setfonts" to "set_fonts" to match the function name within the class.

How?

This PR addresses the issue by updating the doc comment with the correct method name. This was not a code change/will not change any functionality.

Testing Instructions

To test this verify that the updated method name matches the method in the class that is being discussed. The method set_setfonts does not exist and the method is named set_fonts.

Testing Instructions for Keyboard

This is a comment level change and not a visual change on the front end/gutenberg plugin functionality. This keyboard testing instruction may not apply for this small comment change.

Copy link
Contributor

@alexstine alexstine left a comment

Choose a reason for hiding this comment

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

LGTM.

@alexstine alexstine added the [Type] Code Quality Issues or PRs that relate to code quality label Jan 19, 2023
@aristath aristath merged commit 80e7742 into WordPress:trunk Jan 19, 2023
@github-actions github-actions bot added this to the Gutenberg 15.1 milestone Jan 19, 2023
@welenofsky welenofsky deleted the fix/typo-in-webfont-interface-comment branch January 20, 2023 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Code Quality Issues or PRs that relate to code quality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants