Fix typos, spelling mistakes, and documentation errors across repository - #2687
Conversation
|
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 If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message. To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
| ## Reporting Security Issues | ||
|
|
||
| Please see [SECURITY.md](/SECURITY.md). | ||
| Please see [SECURITY.md](./SECURITY.md). |
There was a problem hiding this comment.
Do we need this changes?
There was a problem hiding this comment.
Yes, we should keep this change. ./SECURITY.md is a clearer and more reliable relative link to the security document.
| ## Guidelines | ||
|
|
||
| - As with all WordPress projects, we want to ensure a welcoming environment for everyone. With that in mind, all contributors are expected to follow our [Code of Conduct](https://make.wordpress.org/handbook/community-code-of-conduct/). | ||
| - All WordPress projects are [licensed under the GPLv2+](/LICENSE), and all contributions to the [Performance Lab](https://wordpress.org/plugins/performance-lab/) plugin will be released under the GPLv2+ license. You maintain copyright over any contribution you make, and by submitting a pull request, you are agreeing to release that contribution under the GPLv2+ license. |
There was a problem hiding this comment.
check this one, I think /LICENSE is right already? what do our other WordPress projects use?
There was a problem hiding this comment.
Looks like GitHub's markdown parser already resolves this correctly. Otherwise, this would be a link to https://github.com/LICENSE but no, it links correctly already to https://github.com/WordPress/performance/blob/trunk/LICENSE
In any case, ./LICENSE seems less confusing.
adamsilverstein
left a comment
There was a problem hiding this comment.
left some small feedback
| [Embed Optimizer][5] | `embed-optimizer` | No | [Source][13], [Issues][21], [PRs][29] | ||
| [Enhanced Responsive Images][6] | `auto-sizes` | No | [Source][14], [Issues][22], [PRs][30] | ||
| [Image Placeholders][1] | `dominant-color-images` | No | [Source][9], [Issues][17], [PRs][25] | ||
| [Image Placeholders][1] | `dominant-color-images` | No | [Source][9], [Issues][17], [PRs][25] |

Summary
Files Modified
.github/ISSUE_TEMPLATE/feature_request.md.github/workflows/props-bot.ymlAGENTS.mdCONTRIBUTING.mdREADME.mdplugins/auto-sizes/readme.txtplugins/optimization-detective/detect.jsplugins/optimization-detective/docs/extensions.mdplugins/optimization-detective/docs/hooks.mdplugins/optimization-detective/docs/introduction.mdplugins/webp-uploads/readme.txtAI Tools