* contributing guide -> asciidoc * Update docs/developer/contributing/index.asciidoc Co-authored-by: Peter Schretlen <peter.schretlen@gmail.com> * Update CONTRIBUTING.md Co-authored-by: Peter Schretlen <peter.schretlen@gmail.com> * Update docs/developer/best-practices/stability.asciidoc Co-authored-by: Peter Schretlen <peter.schretlen@gmail.com> * Update docs/developer/contributing/index.asciidoc Co-authored-by: Peter Schretlen <peter.schretlen@gmail.com> * address code review comments * Update docs/developer/contributing/development-documentation.asciidoc Co-authored-by: Peter Schretlen <peter.schretlen@gmail.com> * review comment updates * fix bad ref Co-authored-by: Peter Schretlen <peter.schretlen@gmail.com>
32 lines
1.9 KiB
Text
32 lines
1.9 KiB
Text
[[development-pull-request]]
|
|
=== Submitting a pull request
|
|
|
|
[float]
|
|
==== What Goes Into a Pull Request
|
|
|
|
* Please include an explanation of your changes in your PR description.
|
|
* Links to relevant issues, external resources, or related PRs are very important and useful.
|
|
* Please update any tests that pertain to your code, and add new tests where appropriate.
|
|
* Update or add docs when appropriate. Read more about <<development-documentation>>.
|
|
|
|
[float]
|
|
==== Submitting a Pull Request
|
|
|
|
1. Push your local changes to your forked copy of the repository and submit a pull request.
|
|
2. Describe what your changes do and mention the number of the issue where discussion has taken place, e.g., “Closes #123″.
|
|
3. Assign the `review` and `💝community` label (assuming you are not a member of the Elastic organization). This signals to the team that someone needs to give this attention.
|
|
4. Do *not* assign a version label. Someone from Elastic staff will assign a version label, if necessary, when your Pull Request is ready to be merged.
|
|
5. If you would like someone specific to review your pull request, assign them. Otherwise an Elastic staff member will assign the appropriate person.
|
|
|
|
Always submit your pull against master unless the bug is only present in an older version. If the bug affects both master and another branch say so in your pull.
|
|
|
|
Then sit back and wait. There will probably be discussion about the Pull Request and, if any changes are needed, we'll work with you to get your Pull Request merged into {kib}.
|
|
|
|
[float]
|
|
==== What to expect during the pull request review process
|
|
|
|
Most PRs go through several iterations of feedback and updates. Depending on the scope and complexity of the PR, the process can take weeks. Please
|
|
be patient and understand we hold our code base to a high standard.
|
|
|
|
Check out our <<pr-review>> for our general philosophy for pull request reviews.
|
|
|