Repository navigation
Remove trailing slashes and update redirected Ultralytics URLs - #29
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 10 changed lines across the three files. The URL cleanup is otherwise safe, but the Reddit link still redirects due to subreddit-path casing.
💬 Posted 1 inline comment
- 📝 LOW
README.md:10This new target still redirects: openinghttps://www.reddit.com/r/ultralytics/resolves to Reddit's final/r/Ultralytics/path, so the change does not fully remove the redirected URL described in the PR. Point the badge link at the final canonical target instead. (reddit.com)
UltralyticsAssistant
left a comment
There was a problem hiding this comment.
🔍 PR Review 2
Made with ❤️ by Ultralytics Actions
Since review 1, the Reddit links were corrected to the canonical casing in both README.md and the issue-template configuration, addressing the prior finding. The remaining URL changes are consistent and introduce no regressions. LGTM.
|
🎉 PR #29 has been merged—thank you, @pderrenger!
Your careful URL updates standardize Ultralytics links across the issue templates and |
Summary
Judgment
Validation
🛠️ PR Summary
Made with ❤️ by Ultralytics Actions
🌟 Summary
Updated Ultralytics links across issue templates and
README.mdto remove unnecessary trailing slashes and use Reddit’s canonical HTTPS URL.📊 Key Changes
https://reddit.com/r/ultralyticstohttps://www.reddit.com/r/Ultralytics/..github/ISSUE_TEMPLATE/bug-report.yml,.github/ISSUE_TEMPLATE/config.yml, andREADME.md.🎯 Purpose & Impact