abcxyz / abcxyz/access-on-demand
FR: Ignore Draft PRs
- Dominant language
- Go
- Stars
- 3
- Forks
- 1
- PR merge metrics
- No merged PRs in 30d
Description
### TL;DR
Desired behavior: when a user creates a PR that's in "draft" mode and it's approved, nothing should happen until the PR author marks it as "ready for review".
Current behavior: draft PRs are treated like any other PR, and the AoD request is actuated as soon as the draft PR is reviewed.
### Detailed design
```markdown
```
### Alternatives considered
```markdown
```
### Additional information
_No response_
Contributor guide
Research direction
Look for the GitHub webhook handler or PR status evaluator in the codebase. Identify where PR approvals are processed and where the draft status is checked (or should be). The change likely involves adding a condition to skip actions if the PR is a draft. Check existing tests for PR handling to understand the flow and add a test for the new draft PR behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- github
- Domain
- ci-cd, developer-experience, devops
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100