Standardize .github configuration - #26
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 and the repository contents relevant to link checking. The change only enables the existing Lychee link-check option in Ultralytics Actions, with no conflicting configuration or repository links identified. LGTM.
UltralyticsAssistant
left a comment
There was a problem hiding this comment.
🔍 PR Review 2
Made with ❤️ by Ultralytics Actions
Since the previous review, I also checked the newly added issue forms. The workflow change and issue-form schemas otherwise look sound; one bounded configuration issue remains because triage is referenced but absent from the repository labels. (docs.github.com)
💬 Posted 1 inline comment
- 📝 LOW
.github/ISSUE_TEMPLATE/bug-report.yml:6This template requests atriagelabel, but the repository currently has notriagelabel (it does havebug). GitHub ignores template labels that do not already exist, so submitted bug reports will not receive the intended triage label. Create thetriagelabel in the repository or remove it from this list. ([docs.github.com](https://docs.github.com/en/enterprise-cloud%40latest/communities/using-templates-to-encourage-useful-issues-and-pull-requests/syntax-for-issue-forms?utm_source=openai…
| name: 🐛 Bug Report | ||
| # title: " " | ||
| description: Problems with the SANDD waveform processing tools | ||
| labels: [bug, triage] |
There was a problem hiding this comment.
Resolved at the owner: the triage label now exists in this repository (created org-wide, matching the reference templates in ultralytics/sdk which use labels: [bug, triage]). Template left unchanged.
| name: 🐛 Bug Report | ||
| # title: " " | ||
| description: Problems with the SANDD waveform processing tools | ||
| labels: [bug, triage] |
There was a problem hiding this comment.
Resolved at the owner: the triage label now exists in this repository (created org-wide, matching the reference templates in ultralytics/sdk which use labels: [bug, triage]). Template left unchanged.
UltralyticsAssistant
left a comment
There was a problem hiding this comment.
🔍 PR Review 3
Made with ❤️ by Ultralytics Actions
Since review 2, the earlier triage-label finding was addressed by the owner. I rechecked the Lychee workflow change and all four added issue configuration files; no additional concrete defects remain. LGTM.
|
🎉 PR #26 was successfully merged—thank you, @glenn-jocher!
This PR strengthens the repository’s GitHub configuration with Lychee link checks, structured issue templates, and helpful community links, while keeping blank issues enabled. |
Aligns
.github/with the current Ultralytics repository standard: enables the Lychee broken-link check informat.yml(links: true) and adds the standard issue templates (bug report, feature request, question, config) tailored to this repository.🛠️ PR Summary
Made with ❤️ by Ultralytics Actions
🌟 Summary
Standardized the repository’s GitHub configuration by enabling Lychee link checks and adding structured issue templates for bugs, feature requests, questions, and configuration.
📊 Key Changes
.github/workflows/format.ymlwithlinks: true.🎯 Purpose & Impact