Update issue templates and add auto-assignment for PRs #158
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Refines the GitHub issue template by updating the title prefix from '[FEATURE]' to '[FEATURE REQUEST]' for better clarity and changes the label from 'enhancement' to 'feature' for consistent categorization. Assigns multiple team members as default assignees to ensure proper distribution and tracking of feature requests across the development team.
Assigns specific team members to performance issues for better triage and response times. Updates example hardware specifications to reflect more current systems.
Establishes consistent structure for bug fix pull requests with sections for root cause analysis, reproduction steps, and testing validation. Includes essential checklist items to ensure proper documentation, test coverage, and compatibility considerations are addressed before merging.
Introduces a structured template for feature-related pull requests to ensure consistent documentation of new functionality. The template includes sections for design rationale, implementation details, testing coverage, and documentation updates, helping maintainers review features more effectively.
Provides structured guidance for performance-related pull requests with sections for baseline metrics, optimization approach, benchmark results, and impact assessment. Ensures performance changes include reproducible benchmarks and proper documentation to maintain code quality standards.
Configures GitHub workflow to automatically assign reviewers and assignees when pull requests are opened or marked ready for review. Excludes dependabot PRs from auto-assignment to avoid unnecessary notifications for automated dependency updates. Sets up 8 team members as potential reviewers and assignees to ensure proper code review coverage.
LoserCheems
requested review from
Evanwu1125,
SNHuan,
Thanksyy,
Copilot,
ftgreat,
juliohsu,
wubingheng111 and
zacliu2023
Contributor
There was a problem hiding this comment.
Pull Request Overview
Updates GitHub issue and pull request templates to improve consistency and implements auto-assignment workflow to streamline the contribution process.
- Standardized issue template structure with consistent titles and auto-assignees
- Replaced single PR template with specialized templates for bug fixes, features, and performance
- Added automated reviewer/assignee assignment workflow for new pull requests
Reviewed Changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
.github/workflows/auto-assign.yml |
Implements GitHub workflow for auto-assigning reviewers and assignees |
.github/auto_assign.yml |
Configuration file defining reviewer/assignee lists and assignment rules |
.github/pull_request_template.md |
Removes old comprehensive PR template |
.github/PULL_REQUEST_TEMPLATE/bug_fix.md |
Adds focused template for bug fix PRs |
.github/PULL_REQUEST_TEMPLATE/feature_support.md |
Adds focused template for feature PRs |
.github/PULL_REQUEST_TEMPLATE/performance_optimization.md |
Adds focused template for performance PRs |
.github/ISSUE_TEMPLATE/bug_report.md |
Updates bug report template with consistent formatting and auto-assignees |
.github/ISSUE_TEMPLATE/feature_request.md |
Updates feature request template with consistent formatting and auto-assignees |
.github/ISSUE_TEMPLATE/performance_issue.md |
Updates performance issue template with consistent formatting and auto-assignees |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| - juliohsu | ||
| - wubingheng111 | ||
|
|
||
| numberOfReviewers: 8 |
There was a problem hiding this comment.
The numberOfReviewers is set to 8, but there are only 8 reviewers total in the list. This means all reviewers will always be assigned, which defeats the purpose of having a configurable number. Consider reducing this to 2-4 reviewers to allow for rotation and avoid overwhelming contributors.
Suggested change
| numberOfReviewers: 8 | |
| numberOfReviewers: 3 |
| about: "Introduce a new feature with design context and tests" | ||
| title: "[FEATURE SUPPORT] " | ||
| labels: ["feature"] | ||
|
|
There was a problem hiding this comment.
There's an empty line between the labels and the closing YAML delimiter. This should be removed to maintain consistent YAML formatting across all templates.
Suggested change
| about: "Fix a bug with clear reproduction, scope, and tests" | ||
| title: "[BUG FIX] " | ||
| labels: ["bug"] | ||
|
|
There was a problem hiding this comment.
There's an empty line between the labels and the closing YAML delimiter. This should be removed to maintain consistent YAML formatting across all templates.
Suggested change
LoserCheems
assigned
ftgreat,
Thanksyy,
juliohsu,
SNHuan,
Evanwu1125,
wubingheng111,
LoserCheems,
zacliu2023 and
Copilot
Collaborator
Author
|
No @copilot, I want to give a strong push to the contributors... |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Enhance clarity and consistency of issue templates for bug reports, feature requests, and performance issues. Introduce structured templates for bug fix, feature support, and performance optimization pull requests. Implement auto-assignment of reviewers and assignees to streamline the contribution process. Remove the old pull request template to simplify submissions.