Conversation
There was a problem hiding this comment.
Pull request overview
Updates GitHub automation configs to improve PR auto-labeling and to apply the new A-deps label to Dependabot-generated PRs.
Changes:
- Reworked
.github/labeler.ymllabel definitions into “areas” and “crates”, adding new labels (A-ci,A-docs,A-deps) and updating file globs. - Updated
.github/dependabot.ymlto add default assignees and apply theA-depslabel to Dependabot PRs.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| .github/labeler.yml | Adds/renames label rules and updates glob patterns for auto-labeling. |
| .github/dependabot.yml | Adds assignees and A-deps label to Dependabot update PRs. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
.github/dependabot.yml
Outdated
| assignees: | ||
| - cot-rs/maintainers | ||
| labels: | ||
| - A-deps |
There was a problem hiding this comment.
Same issue here: assignees expects individual GitHub usernames; an org/team slug like cot-rs/maintainers may break or be ignored by Dependabot. Prefer reviewers for a team, or list usernames under assignees.
There was a problem hiding this comment.
@copilot provide docs pages that show reviewers is a valid field
|
Bencher Report| Branch | automatic-labeler |
| Testbed | github-ubuntu-latest |
Click to view all benchmark results
| Benchmark | Latency | Benchmark Result milliseconds (ms) (Result Δ%) | Upper Boundary milliseconds (ms) (Limit %) |
|---|---|---|---|
| empty_router/empty_router | 📈 view plot 🚷 view threshold | 6.43 ms(+7.58%)Baseline: 5.98 ms | 7.17 ms (89.69%) |
| json_api/json_api | 📈 view plot 🚷 view threshold | 1.16 ms(+12.31%)Baseline: 1.03 ms | 1.19 ms (97.40%) |
| nested_routers/nested_routers | 📈 view plot 🚷 view threshold | 1.07 ms(+12.17%)Baseline: 0.95 ms | 1.09 ms (97.78%) |
| single_root_route/single_root_route | 📈 view plot 🚷 view threshold | 1.03 ms(+13.24%)Baseline: 0.91 ms | 1.05 ms (98.42%) |
| single_root_route_burst/single_root_route_burst | 📈 view plot 🚷 view threshold | 17.71 ms(+0.56%)Baseline: 17.61 ms | 21.14 ms (83.76%) |
seqre
force-pushed
the
automatic-labeler
branch
from
be5ad98 to
36c3088
Compare
seqre
force-pushed
the
automatic-labeler
branch
from
36c3088 to
778e5e4
Compare
This comment was marked as spam.
This comment was marked as spam.
Codecov Report✅ All modified and coverable lines are covered by tests.
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
Co-authored-by: Marek Grzelak <git@seqre.dev>
Description
Improve automatic labeling and align it with new labels.
Type of change