open-telemetry / open-telemetry/shared-workflows
PR Dashboard: stop using Outdated as an implicit reason to route thread back to reviewer
Open
Nobody has claimed this yet.
- Dominant language
- No language data
- Stars
- 7
- Forks
- 18
- Avg merge
- 22h 1m
- Merged PRs (30d)
- 75
Description
After implementing
We could consider removing "Outdated" as being an implicit reason to route thread back to reviewer and instead require an explicit reply (or explicit resolve discussion).
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reading issue #26 and locating the PR Dashboard implementation that routes review threads. Confirm how "Outdated" currently triggers routing, then change the behavior so only an explicit reply or explicit discussion resolution routes a thread back to the reviewer. Done means outdated status alone no longer causes that routing and the relevant workflow checks pass.
Written by the indexing model from the issue text.
Assessment
- Domain
- ci-cd
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100