Standardize .github configuration - #1324
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 .github/workflows/format.yml. The issues: opened trigger and issues: write permission correctly enable Ultralytics Actions to label newly opened issues without regressing the existing pull-request workflow. LGTM.
UltralyticsAssistant
left a comment
There was a problem hiding this comment.
🔍 PR Review 2
Made with ❤️ by Ultralytics Actions
Review 2: the previous format.yml change remains addressed. The newly added issue intake introduces stale HUB support guidance and a missing default label; these issues should be corrected before merge.
💬 Posted 2 inline comments
- 💡 MEDIUM
.github/ISSUE_TEMPLATE/config.yml:3AGENTS.md says this repository is historical and HUB shut down on July 31, 2026; it contains only Platform redirects/workflows, and HUB APIs/services no longer work. Enabling blank issues together with the three new HUB-branded forms creates a live-looking support intake for an unsupported product, directing users to report HUB bugs, features, and API issues here. Retarget the intake to Platform, or leave issue creation disabled. - 📝 LOW
.github/ISSUE_TEMPLATE/bug-report.yml:6The repository's current label set does not containtriage, and GitHub does not create missing labels when applying issue-form defaults. Bug reports from this form will therefore silently receivebugbut nottriage, defeating the intended triage behavior. Add the label in repository settings or remove it from the form. (github.com)
UltralyticsAssistant
left a comment
There was a problem hiding this comment.
🔍 PR Review 3
Made with ❤️ by Ultralytics Actions
Since review 2, the forms now disclose HUB's retirement and add a Platform SDK contact link; the workflow change remains addressed. Two issues remain unresolved: this historical repository still exposes HUB issue intake and blank issues, and the bug form still references a missing triage label. Not LGTM.
💬 Posted 2 inline comments
- 💡 MEDIUM
.github/ISSUE_TEMPLATE/config.yml:3Although the forms now mention the retirement date, this still enables a live HUB support intake: visitors can submit blank issues or use HUB-branded bug, feature, and question forms whose components describe the retired web app. This repository is now historical and contains only redirects/workflows, so these submissions have no supported owner after July 31, 2026. Disable issue creation (and remove the forms), or retarget the entire intake to a currently supported repository. - 📝 LOW
.github/ISSUE_TEMPLATE/bug-report.yml:6triageis still not an existing repository label, so bug reports created from this form will receivebugbut silently omittriage; GitHub does not automatically add labels that do not already exist. Removetriagehere or create the label in the repository settings.
|
Re Review 3: both remaining findings are intentional per the org standard — every Ultralytics public repository carries the full issue template set with blank issues enabled; the forms disclose HUB's retirement and route active Platform support to the Platform API docs and ultralytics/sdk issues, and remain for issues with this historical repository itself. The |
UltralyticsAssistant
left a comment
There was a problem hiding this comment.
🔍 PR Review 4
Made with ❤️ by Ultralytics Actions
Reviewed all five changed files. The workflow changes remain correct, and the earlier HUB-intake and triage concerns are settled by the author’s clarification. One remaining issue is that the new Platform SDK support link leads to a repository where issue creation is restricted, so the advertised reporting route does not work. (github.com)
💬 Posted 1 inline comment
- 📝 LOW
.github/ISSUE_TEMPLATE/config.yml:12This newly added destination cannot accept reports: theultralytics/sdkIssues page currently states that issue creation is restricted. Users following this link—or the same link repeated in all three forms—will be unable to file the promised active Platform SDK bug or feature report. Replace every occurrence with a support destination that accepts issues, or remove this contact link. (github.com)
|
Re Review 4: the ultralytics/sdk destination is the mandated Platform SDK support route (its own issue templates and Platform API reference point there). The "issue creation is restricted" banner is a repository setting on ultralytics/sdk, not something this template can fix; flagged for the repository owner to lift there. |
UltralyticsAssistant
left a comment
There was a problem hiding this comment.
🔍 PR Review 5
Made with ❤️ by Ultralytics Actions
Since review 4, no new repository-local defect or regression is introduced in the five-file diff. The workflow trigger/permission change and issue-template configuration remain consistent with the stated org standard, and the previously discussed external SDK destination concern is outside this PR's files. LGTM.
|
🎉 Thank you, @glenn-jocher, for the merged PR #1324!
This PR brings together HUB-specific issue forms, blank issue support, helpful contact links, and updated automation triggers to create a more structured and accessible issue intake experience. |
issues: openedtrigger andissues: writepermission toformat.ymlso Ultralytics Actions labels new issues, matching the org standard.config.yml.🛠️ PR Summary
Made with ❤️ by Ultralytics Actions
🌟 Summary
Standardized the repository’s issue intake and automation by adding HUB-specific issue templates, enabling blank issues, and allowing the Actions workflow to process newly opened issues.
📊 Key Changes
ultralytics/sdk.format.ymlto trigger on newly opened issues and grantissues: writepermission for issue comments and labels.🎯 Purpose & Impact