diff options
author | Teemu Suo-Anttila <tsuoanttila@users.noreply.github.com> | 2017-10-23 15:34:42 +0300 |
---|---|---|
committer | Péter Török <31210544+torok-peter@users.noreply.github.com> | 2017-10-23 15:34:42 +0300 |
commit | ebc765c1a78c5facaf5e65122e317da0af6ca472 (patch) | |
tree | 30290c1cc4402015d58fd5fc2f918d2bdda3652a | |
parent | d0dde67c186f624b9e1ddaa6cfd03ae4d41b2d64 (diff) | |
download | vaadin-framework-ebc765c1a78c5facaf5e65122e317da0af6ca472.tar.gz vaadin-framework-ebc765c1a78c5facaf5e65122e317da0af6ca472.zip |
Add contribution guideline (#10083)
* Add contribution guideline
* Make suggested improvements to CONTRIBUTING.md
* Add link to good first issues and project setup
* Small update to CONTRIBUTING.md
-rw-r--r-- | .github/CONTRIBUTING.md | 34 |
1 files changed, 34 insertions, 0 deletions
diff --git a/.github/CONTRIBUTING.md b/.github/CONTRIBUTING.md new file mode 100644 index 0000000000..cbae52717d --- /dev/null +++ b/.github/CONTRIBUTING.md @@ -0,0 +1,34 @@ +# Vaadin Framework Contribution Guidelines + +## Project setup + +See the repository [root level README](https://github.com/vaadin/framework/blob/master/README.md) for instructions on setting up the environment to keep up the formatting and code style. + +## Pull Requests + +### Making your PR + +A new pull request should start by forking the Vaadin Framework project (or checking that your fork is up-to-date). + +Unless the issue is specific to a single version of the Framework, you should always aim to make the patch on top of the `master` branch. + +In your repo, make a new branch from `master`. Make your code changes in that branch, and once you're done make a pull request to the `vaadin/framework` branch `master`. + +We recommend making sure that the "Allow edits from maintainers" checkbox on the pull request page is checked. This allows us to commit minor fixes, like correcting typos, without bothering you. + +### Getting feedback and responding to it + +Once the pull request has been opened, it will be reviewed by the development team behind Vaadin Framework. We pay close attention to the readability of the code as well as documentation and correctness. + +If this was your first pull request to the Vaadin repository, you will also have to sign our CLA. This needs to be done with the same user you use to make the commits. If you're using multiple e-mails and GitHub users, each will need to sign the CLA. + +## Issues + +Only use GitHub issues for bugs and feature requests. For general support from the community, see https://vaadin.com/forum + +For bugs, please provide at minimum the following information: +- Vaadin version +- Description of the bug +- If possible, minimal reproducible example + +We use the label `good first issue` to mark easy issues to get started with. The list is available [here](https://github.com/vaadin/framework/issues?q=is%3Aissue+is%3Aopen+label%3A%22good+first+issue%22). |