Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create "Draft PR" by default option #2238

Closed
Daenyth opened this issue Jul 15, 2019 · 12 comments
Closed

Create "Draft PR" by default option #2238

Daenyth opened this issue Jul 15, 2019 · 12 comments

Comments

@Daenyth
Copy link

Daenyth commented Jul 15, 2019

I'd like it if, when I create a pull request, the page automatically had "Create Draft PR" as the default option, instead of needing to manually select it myself.

I'd be happy if this is a configuration toggle, since I imagine not everyone would want that behavior.

@fregante
Copy link
Member

fregante commented Jul 19, 2019

I think people will just end up creating Draft PRs and forgetting about it: they wait for reviews and reviewers wait for “ready” status.

It’s have to come from GitHub so more people are aware of this behavior

@sindresorhus
Copy link
Member

I don’t think that’s a good default.

@Daenyth
Copy link
Author

Daenyth commented Jul 19, 2019

@bfred-it is there a reason not to make it configurable? At my work we use draft PRs heavily, it would be convenient to streamline. Github makes it impossible to go from non-draft to draft status, so creating it the wrong way by accident isn't fixable (except for closing the pr and opening a new one), while creating a draft one when you meant non-draft is fixable with one button

@djbingham
Copy link

I would love to be able to make "draft" the default state for PRs I open. We have a policy of creating a PR as soon as we start a piece of work so that the work is visible. However, as people frequently miss the opportunity to mark it as a Draft PR, we have no obvious way to distinguish which PRs are ready for review and which are work-in-progress.

I don’t think that’s a good default.

As @Daenyth explained after that comment , it's easy to switch from draft to non-draft but impossible to switch the other way. To my mind, that means draft is the only default that makes sense. It's also completely logical that when a PR is first created, it's likely to be a draft. When it's ready for review, it should then be promoted to that state.

I do see a reasonable case that people may be confused by a change in the default behaviour. Perhaps a good compromise would be enabling a PR to be switched to draft state after it's created, then the default would not matter so much. It might also help make the draft button less forgettable if it was visible when creating a PR, rather than hidden behind a menu.

I think people will just end up creating Draft PRs and forgetting about it

That seems quite unlikely to me, especially if the preference is configurable. I find the visual difference between a draft and non-draft PR pretty obvious when I create them. Also, I would have thought someone submitting a PR would take an interest in how it progresses. Do most people not follow up their PRs if they haven't heard anything for a while? If a PR is hanging around in a draft state, people can always message the author to find out why.

@busches
Copy link
Member

busches commented Oct 3, 2019

#2473 seems like a good compromise.

@Daenyth
Copy link
Author

Daenyth commented Oct 3, 2019

Perhaps a good compromise would be enabling a PR to be switched to draft state after it's created,

Unfortunately only Github themselves is able to make this change

@grebenyuksv-preply
Copy link

That would be a great default for us because we keep asking code owners for code review accidentally and automatically

@fregante
Copy link
Member

fregante commented Nov 11, 2019

We won't change the default and #2473 is already here: The "Create draft" button is very visible and I noticed I myself create more drafts thanks to that feature.

2019-10-02 at 17 33

@Daenyth
Copy link
Author

Daenyth commented Nov 11, 2019

I think the title I posted this with is unfortunate.

It would be better as:
Create configuration option which allows "create draft" to be displayed as the "Default" button on the "create PR" page.
The option can default to "off", which means that the behavior of the plugin as of today wouldn't change - this would be in there only for folks who wanted it.
Alternately, default to "on" for new extension installs, but if the extension was updated from an older version, set the config to "off" so as to not confuse existing users

@muescha
Copy link
Contributor

muescha commented Nov 30, 2019

@Daenyth with #2486 the DropDown Button is now I. 2 seperate Buttons. With Stylish you can hide with css the Button you don't like to see

@busches
Copy link
Member

busches commented Dec 4, 2019 via email

@refined-github refined-github locked as resolved and limited conversation to collaborators Mar 30, 2020
@fregante
Copy link
Member

fregante commented Apr 9, 2020

Perhaps a good compromise would be enabling a PR to be switched to draft state after it's created

They did! https://github.blog/changelog/2020-04-08-convert-pull-request-to-draft/

Convert pull request to draft

Any user with write access to the repository can convert a pull request to a draft pull request.

To convert a pull request to a draft, look for the “Convert to draft” link in the Reviewers section on the pull request’s Conversations tab. Clicking the link displays a modal asking you to confirm the change.

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

No branches or pull requests

7 participants