Repository navigation
Remove trailing slashes and update redirected Ultralytics URLs - #48
Conversation
|
👋 Hello @pderrenger, 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 all 15 changed files and the full URL-only diff. The changes are limited to canonicalizing Ultralytics URLs and updating the Reddit redirect without altering runtime logic, metadata structure, or workflow behavior. No concrete bugs or regressions found. LGTM.
|
🎉 Thank you, @pderrenger, and congratulations on the merged PR!
This PR puts that principle into practice by canonicalizing 72 Ultralytics URLs across 15 files, reducing redirects and keeping links consistent without changing model or workflow logic. |
Canonicalizes every Ultralytics URL in the repo to its no-trailing-slash form and refreshes one stale redirect.
Changes
Trailing slashes — 72 URLs across 15 files
Removed the terminating path slash from every
ultralytics.com/*.ultralytics.comURL. Only slashes that terminate a path were touched; path separators were never affected.https://docs.ultralytics.com/→https://docs.ultralytics.com,.../models/yoloe/→.../models/yoloereplicate_metadata.pypaper_urlSPECS entries (/tasks/segment/→/tasks/segment).github/ISSUE_TEMPLATE/*config.ymlDocs/Forum contact links,minimum-reproducible-example/→...-exampleREADME.md<a href>and Forums badge targetAll rewritten destinations were verified live and return HTTP 200 with no redirect.
Redirect refresh — 1 accepted, 0 rejected
https://reddit.com/r/ultralytics→https://www.reddit.com/r/ultralytics/(root README Reddit badge). Permanent host canonicalization, and it matches the form already used in all nine model READMEs. This is the only URL in the repo where the scan proposed a change; the other 116 unique URLs resolved cleanly.No dead links were reported by the scan.
Judgment calls (deliberately left unchanged)
https://ultralytics.com/discordin.github/ISSUE_TEMPLATE/config.yml— vanity shortlink meant to stay a redirect (its target is an expiring invite); it carries no trailing slash anyway.server=https%3A%2F%2Fcommunity.ultralytics.cominside the shields.io Forums badge — URL-encoded parameter, kept byte-identical; only the badge's link target was de-slashed.https://ultralytics.com/licensefile headers — bot-owned strings written by Ultralytics Actions; already slash-free.https://ultralytics.com/images/bus.jpgin the CLI examples — a file path, not a directory terminator.cocodataset.org/,pytorch.org/get-started/locally/,pypi.org/project/ultralytics/) are out of scope for the slash sweep and were left as-is..github/workflows/push.ymlis flagged by Prettier as unformatted, but that is pre-existing and untouched by this PR.Validation
/#fragmentand/?query, sentence-finalcom/.andcom/!, markdown/HTML/backtick delimiters, and negatives forgithub.com/ultralytics/..., emails,notultralytics.com,%3A%2F%2F-encoded params,{placeholder}templates, f-strings, and the ellipsis/....python -m py_compileon both scripts,replicate_metadata.py --dry-run(all nine models), YAML parse of everycog.yamland.github/**/*.yml,ruff format --check+ruff checkclean,prettier@3.8.5 --print-width 120 --checkclean on every changed file.The diff is URL text only — no logic, formatting, or content changes.
🛠️ PR Summary
Made with ❤️ by Ultralytics Actions
🌟 Summary
Canonicalized 72 Ultralytics URLs across 15 files by removing trailing slashes and updated the root README Reddit link to its
wwwcanonical URL.📊 Key Changes
paper_urlentries inreplicate_metadata.pyto slash-free documentation paths..github/ISSUE_TEMPLATEfiles and the rootREADME.md.reddit.com/r/ultralyticstowww.reddit.com/r/ultralytics/.🎯 Purpose & Impact