Remove trailing slashes and update redirected Ultralytics URLs - #28
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 three changed files. The trailing-slash removals are otherwise behavior-preserving, but the Reddit URL update does not eliminate the redirect and leaves a second outdated reference in the issue-template configuration. (reddit.com)
💬 Posted 1 inline comment
- 📝 LOW
README.md:8This target still redirects:https://www.reddit.com/r/ultralytics/resolves to Reddit's canonicalhttps://www.reddit.com/r/Ultralytics/. The issue-template link in.github/ISSUE_TEMPLATE/config.ymlalso remainshttps://reddit.com/r/ultralytics, so the redirect cleanup is incomplete. Update both references to the final canonical URL. (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 previously reported Reddit redirect issue has been addressed in both README.md and .github/ISSUE_TEMPLATE/config.yml. The remaining trailing-slash removals are limited to the affected documentation and configuration URLs, while the canonical Reddit URL retains its required trailing slash. No behavioral or compatibility issues found. LGTM.
UltralyticsAssistant
left a comment
There was a problem hiding this comment.
🔍 PR Review 3
Made with ❤️ by Ultralytics Actions
Since review 2, the previously reported Reddit canonical-link issue remains addressed in both README.md and the issue-template configuration. The current diff only updates the intended documentation and contact URLs; no behavioral, compatibility, or maintainability issues remain. LGTM.
UltralyticsAssistant
left a comment
There was a problem hiding this comment.
🔍 PR Review 4
Made with ❤️ by Ultralytics Actions
Since the last review, the previously reported Reddit canonical-link issue remains resolved, and the current diff only normalizes the intended URLs plus updates the MSVM publication link. The changed links are correctly used in their README and issue-template contexts, with no behavioral, compatibility, or maintainability defects found. LGTM.
|
🎉 PR #28 has been merged—thank you, @pderrenger!
This PR brings together important documentation and contact-link updates, including canonical Ultralytics, Community Forum, Reddit, and MSVM publication destinations, while keeping MSVM functionality unchanged. |
Summary
Judgment
Validation
🛠️ PR Summary
Made with ❤️ by Ultralytics Actions
🌟 Summary
Updated Ultralytics, Community Forum, Reddit, and MSVM publication links to remove unnecessary trailing slashes and use their current canonical destinations.
📊 Key Changes
https://community.ultralytics.com.https://www.reddit.com/r/Ultralytics/address.🎯 Purpose & Impact