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

Fix several typos and spelling mistakes for 6.6 #6159

Closed
wants to merge 18 commits into from

Conversation

swissspidy
Copy link
Member

@swissspidy swissspidy commented Feb 22, 2024

I used a GitHub action to find some spelling mistakes / typos and then manually fixed all of them.

The intention is to fix them all in one go, but not actually commit the GitHub action itself.

Trac ticket:


This Pull Request is for code review only. Please keep all other discussion in the Trac ticket. Do not merge this Pull Request. See GitHub Pull Requests for Code Review in the Core Handbook for more details.

worais

This comment was marked as resolved.

@swissspidy

This comment was marked as resolved.

@worais

This comment was marked as resolved.

@swissspidy swissspidy changed the title Test spell check action Fix several typos and spelling mistakes for 6.6 Apr 12, 2024
@swissspidy swissspidy marked this pull request as ready for review April 12, 2024 14:10
Copy link

github-actions bot commented Apr 12, 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.

Core Committers: Use this line as a base for the props when committing in SVN:

Props swissspidy, jucaduca, sergeybiryukov.

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

@swissspidy
Copy link
Member Author

swissspidy commented Apr 12, 2024

@SergeyBiryukov Would appreciate your review here as you're usually the one making this kind of changes.

My idea is to fix the typos as part of https://core.trac.wordpress.org/ticket/60699. The GHA is just for testing, not for committing.

This comment was marked as resolved.

@SergeyBiryukov
Copy link
Member

Would appreciate your review here as you're usually the one making this kind of changes.

Curious why wp-admin/includes/ms.php and wp-includes/formatting.php are excluded. Looks great to me otherwise! 👍

@swissspidy
Copy link
Member Author

There were some false positives reported in those files, so it was easier to just ignore them here during development.

@swissspidy
Copy link
Member Author

Committed in https://core.trac.wordpress.org/changeset/57987 / https://core.trac.wordpress.org/changeset/57988

Can do this again periodically I think

@swissspidy swissspidy closed this Apr 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants