abcxyz / abcxyz/access-on-demand

FR: Ignore Draft PRs

Open
#173 0 comments 0 reactions 0 assignees View on GitHub
enhancement
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.