Move the pull request guide to the developer guide
Motivation: CONTRIBUTING.md is useful only because it lets Github show a user the link to it so the user can check what information we need before submitting a bug report. However, Github does not do the same for a pull request submission form, and thus there's no reason to keep the information about how to submit a good pull request in CONTRIBUTING.md. Modification: Replace the section about issuing a pull request with the link to the official developer guide. Result: CONTRIBUTING.md is easier to maintain.
This commit is contained in:
parent
722f6c5e1a
commit
e8bdaff80f
@ -40,12 +40,7 @@ Linux infinity 3.10.32-1-lts #1 SMP Sun Feb 23 09:44:24 CET 2014 x86_64 GNU/Linu
|
|||||||
My system has IPv6 disabled.
|
My system has IPv6 disabled.
|
||||||
```
|
```
|
||||||
|
|
||||||
## How to submit a pull request
|
## How to contribute your work
|
||||||
|
|
||||||
Pull requests should be targeted at the branch for the latest stable releases. If the pull request is for fixing a bug which also affects an old branch like `3.x`, we recommend you to submit another pull request for that branch, too.
|
Before submitting a pull request or push a commit, please read [our developer guide](http://netty.io/wiki/developer-guide.html).
|
||||||
|
|
||||||
1. [Rebase](http://git-scm.com/book/en/Git-Branching-Rebasing) your changes against the upstream branch. Resolve any conflicts that arise.
|
|
||||||
1. Write JUnit test cases if possible. If not sure about how to write one, ask to write one before it's merged.
|
|
||||||
1. Run `mvn test` before the initial submission or the subsequent pushes, and ensure the build succeeds.
|
|
||||||
|
|
||||||
For more information on developing Netty, please refer to [the developer guide](http://netty.io/wiki/developer-guide.html).
|
|
||||||
|
Loading…
Reference in New Issue
Block a user