Repository navigation
Preserve proper names in website spellcheck - #377
Conversation
|
👋 Hello @glenn-jocher, thank you for submitting a
For more guidance, please refer to our Contributing Guide. Don't hesitate to leave a comment if you have any questions. Thank you for contributing to Ultralytics! 🚀 |
UltralyticsAssistant
left a comment
There was a problem hiding this comment.
🔍 PR Review
Made with ❤️ by Ultralytics Actions
Reviewed the spelling-check workflow change. The exclusions are limited to the two exact, case-sensitive, word-bounded phrases, while retaining the .alls exception and the existing British-spelling checks. LGTM.
|
🎉 This PR has been merged—thank you, @glenn-jocher!
The targeted exceptions preserve the existing British-English checks while allowing these two proper names. |
Website spellcheck reports
Labour ==> Laborin International Labour Organization andStorey ==> Storyin Margaret-Anne Storey. Both are correct proper names in the CMS and on the published pages.Exempt only these exact, case-sensitive, word-bounded phrases while retaining
en-GB_to_en-USand the existing.allsexception. Use the existing codespell multiline exclusion option so matching happens before URI stripping, which otherwise consumesInternationalimmediately after an HTML link URL. Ordinary British spellings, includingLabourandStoreyelsewhere on the same line, remain errors. No CMS content changes.Validation:
labour,storey,colour,organise,Labour, andStorey..allsremains excluded.git diff --checkpass.🛠️ PR Summary
Made with ❤️ by Ultralytics Actions
🌟 Summary
The website spellcheck now exempts two exact proper-name phrases without suppressing British spelling errors elsewhere.
📊 Key Changes
.allsand the phrasesInternational Labour OrganizationandMargaret-Anne Storey.en-GB_to_en-USspelling checks, so standaloneLabourandStoreyremain flagged.🎯 Purpose & Impact
.allsexception remain in place.