-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
The grammatical issue has been fixed in option.php #4466
The grammatical issue has been fixed in option.php #4466
Conversation
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 @nazmulhudadev, There is one more instance https://github.com/WordPress/wordpress-develop/blob/trunk/src/wp-includes/option.php#L730 that need to be correct.
Hi @mukeshpanchal27 Thanks for reviewing the changes and other instances. The mistake of [line 730](https://github.com/WordPress/wordpress-develop/blob/trunk/src/wp-includes/option.php#L730) has been corrected as well.
Hi @mukeshpanchal27, Thanks for reviewing the changes. I've made the required changes in line 730 as well. |
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 @nazmulhudadev, The changes look good to me.
@nazmulhudadev Found one more instance on https://github.com/krupal-panchal/wordpress-develop/blob/trunk/src/wp-includes/ms-blogs.php#L422 Also, address this. |
Hi @krupal-panchal, Thanks for spotting the typo. |
Merge the changes in #4480 into this and close that PR. |
committed in https://core.trac.wordpress.org/changeset/55884 |
The mistake has been fixed.
Trac ticket: https://core.trac.wordpress.org/ticket/58338