Skip to content
This repository has been archived by the owner on May 20, 2023. It is now read-only.

Exclude Draft PRs #319

Closed
wants to merge 5 commits into from
Closed

Exclude Draft PRs #319

wants to merge 5 commits into from

Conversation

theaccordance
Copy link

@theaccordance theaccordance commented Jan 21, 2021

This Pull Requests adds exemptDrafts option to exclude Draft pull requests.

Related Issue: #265

giphy


View rendered README.md

@theaccordance theaccordance changed the title Jm/exclude drafts Exclude Draft PRs Jan 21, 2021
/** Can't search issues with draft:false in the query so this is appended only for pulls **/
if (type === 'pulls') {
queryParts.push(exemptDrafts ? 'draft:false' : '')
}
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't write tests for this yet, but if this PR is agreeable I will definitely add those.

As for why I conditionally wrapped it - if you include draft:boolean in your query for issues, nothing returns - so this can only be applied for pulls.

@stale
Copy link

stale bot commented May 10, 2021

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

@stale stale bot added the wontfix label May 10, 2021
Copy link

@dantwining dantwining left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FWIW, to me, as some random person on the internet, this seems like the right way to implement this. I'm not sure how we get traction from the maintainers to approve/merge this?

@stale stale bot removed the wontfix label Jun 15, 2021
@theaccordance
Copy link
Author

@bkeepers I realize your bio says "formerly github" but is there someone you can nudge about this pull request? I'm gonna close it otherwise and find an alternative solution.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants