Skip to content

Commit aa457df

Browse files
committed
minor #15023 Remove comment about PR draft status (Nyholm)
This PR was submitted for the 5.x branch but it was merged into the 4.4 branch instead. Discussion ---------- Remove comment about PR draft status We should not encourage adding draft PRs. We do however encourage to submit a PR that are in WIP. (Ie just added code, not tests) Commits ------- f68b17f Remove comment about PR draft status
2 parents 6e716e1 + f68b17f commit aa457df

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

contributing/code/pull_requests.rst

+1-3
Original file line numberDiff line numberDiff line change
@@ -370,8 +370,7 @@ because you want early feedback on your work, add an item to todo-list:
370370
- [ ] gather feedback for my changes
371371
372372
As long as you have items in the todo-list, please prefix the pull request
373-
title with "[WIP]". If you do not yet want to trigger the automated tests,
374-
you can also set the PR to `draft status`_.
373+
title with "[WIP]".
375374

376375
In the pull request description, give as much detail as possible about your
377376
changes (don't hesitate to give code examples to illustrate your points). If
@@ -436,4 +435,3 @@ before merging.
436435
.. _`searching on GitHub`: https://github.com/symfony/symfony/issues?q=+is%3Aopen+
437436
.. _`Symfony Slack`: https://symfony.com/slack-invite
438437
.. _`Travis-CI`: https://travis-ci.org/symfony/symfony
439-
.. _`draft status`: https://help.github.com/github/collaborating-with-issues-and-pull-requests/about-pull-requests#draft-pull-requests

0 commit comments

Comments
 (0)