IntenseFinance/neovim
3
0
Fork 0
mirror of https://github.com/neovim/neovim.git synced 2025-02-25 18:55:25 -06:00
Code Issues Packages Projects Releases Wiki Activity

docs: update pull request workflow #15243

Browse Source 
The WIP, RFC and RDY stage workflow is outdated and unnecessary. The new
workflow utilize the draft pull request on GitHub.
This commit is contained in:
dundargoc 2021-08-17 14:58:49 +02:00 committed by GitHub
parent 2cb8db34e3
commit 35c6aef1c9
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 17 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
CONTRIBUTING.md
View File

@ -42,7 +42,7 @@ Developer guidelines
Pull requests (PRs) Pull requests (PRs)
--------------------- ---------------------
- To avoid duplicate work, create a `[WIP]` pull request as soon as possible. - To avoid duplicate work, create a draft pull request as soon as possible.
- Your PR must include **test coverage.** See [test/README.md][run-tests]. - Your PR must include **test coverage.** See [test/README.md][run-tests].
- Avoid cosmetic changes to unrelated files in the same commit. - Avoid cosmetic changes to unrelated files in the same commit.
- Use a [feature branch][git-feature-branch] instead of the master branch. - Use a [feature branch][git-feature-branch] instead of the master branch.
@ -62,16 +62,22 @@ Pull requests (PRs)
- During a squash/fixup, use `exec make -C build unittest` between each - During a squash/fixup, use `exec make -C build unittest` between each
pick/edit/reword. pick/edit/reword.
### Stages: WIP, RFC, RDY ### Stages: Draft and Ready for review
Pull requests have three stages: `[WIP]` (Work In Progress), `[RFC]` (Request Pull requests have two stages: Draft and Ready for review.
For Comment) and `[RDY]` (Ready).
1. `[RFC]` is assumed by default, **do not** put "RFC" in the PR title (it adds 1. [Create a Draft PR][pr-draft] while you are _not_ requesting feedback as
noise to merge commit messages). you are still working on the PR.
2. Add `[WIP]` to the PR title if you are _not_ requesting feedback and the work 2. [Change your PR to ready][pr-ready] when the PR is ready for review.
is still in flux.
3. Add `[RDY]` to the PR title if you are _done_ and only waiting on merge. You can convert the state of your PR back to Draft (or Ready for review) at any
time. You can also skip the Draft stage if your PR is ready for review from the
beginning.
Do __not__ add any labels like `[RFC]` or `[WIP]` in the title to indicate the
state of your PR: this just adds noise. Non-Draft PRs are assumed to be open
for comments by default; if you want feedback from specific people, `@`-ping
them in a comment.
### Commit messages ### Commit messages
@ -284,3 +290,5 @@ as context, use the `-W` argument as well.
[complexity:low]: https://github.com/neovim/neovim/issues?q=is%3Aopen+is%3Aissue+label%3Acomplexity%3Alow [complexity:low]: https://github.com/neovim/neovim/issues?q=is%3Aopen+is%3Aissue+label%3Acomplexity%3Alow
[master error list]: https://raw.githubusercontent.com/neovim/doc/gh-pages/reports/clint/errors.json [master error list]: https://raw.githubusercontent.com/neovim/doc/gh-pages/reports/clint/errors.json
[wiki-contribute-help]: https://github.com/neovim/neovim/wiki/contribute-%3Ahelp [wiki-contribute-help]: https://github.com/neovim/neovim/wiki/contribute-%3Ahelp
[pr-draft]: https://docs.github.com/en/github/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request
[pr-ready]: https://docs.github.com/en/github/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/changing-the-stage-of-a-pull-request