Repository navigation
Fix website glossary spellcheck false positives - #369
Conversation
…ce.mit.edu, glossary terms) Ignore proper nouns, a code variable, and US-standard "dialogue" flagged on www glossary pages; exclude bot-protected uber.com and dspace.mit.edu from lychee. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
👋 Hello @nastijakh, 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 codespell ignore-list additions and the bot-protected-domain exclusions across all three lychee calls. The exclusions are synchronized and the changes address the reported false positives without an evident regression. LGTM; the spelling allowlist remains global to the rendered HTML spellcheck.
|
🎉 PR #369 has been merged! Thank you @nastijakh, with contributions from @glenn-jocher, for updating the spellcheck ignore list and fixing the Feature Store and Spatial Computing links.
These careful fixes help keep the glossary links current and prevent the reported spellcheck false positives. |
Fixes the 21 spelling false positives in the scheduled Website links and spellcheck run.
Adds 11 verified terms to the existing rendered-HTML codespell ignore list:
fpr,dialogue,dialogues,SME,redner,sEH,Mata,IST,DifferNet,Sclar, andCentre. These are code identifiers, technical names, proper nouns, or accepted US English spellings. The change is one line in the existing spelling owner.The two link failures were repaired directly in the published CMS records through the Portal API, with before/after backups:
No new domain exclusions remain; Uber and MIT links stay covered by the checker.
Validation: codespell 2.4.3 with the workflow's exact options reproduced all 21 errors across 18 live pages on the base configuration, then reported zero with this change. Control misspellings remained detectable. Both replacement URLs passed lychee without exclusions, and both public glossary pages now render the new links with the old links absent. Both workflow files parse as YAML, Prettier passes, and all three existing link-check exclusion patterns remain identical and unchanged from main.
🛠️ PR Summary
Made with ❤️ by Ultralytics Actions
🌟 Summary
Updated the website spellcheck ignore list and repaired two glossary links to resolve the scheduled checker’s reported issues.
📊 Key Changes
🎯 Purpose & Impact