Updated CONTRIBUTING.md (#1139)

* Updated CONTRIBUTING.md

Added "Customs around integration, branching, tagging, and releases" section to contribution guide.

* Clarify that the admonition to squash PRs is directed at integrators
accellbaker
Ali-RS 5 years ago committed by GitHub
parent 57db8f618f
commit 02ccce08ec
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 9
      CONTRIBUTING.md

@ -22,6 +22,15 @@ p.s. We will try hold ourselves to a [certain standard](http://www.defmacro.org/
Developers in the Contributors team can push directly to Main instead of submitting pull requests, however for new features it is often a good idea to do a pull request as a means to get a last code review. Developers in the Contributors team can push directly to Main instead of submitting pull requests, however for new features it is often a good idea to do a pull request as a means to get a last code review.
## Customs around integration, branching, tagging, and releases
- Most pull requests are integrated directly into the master branch of the repository.
- Integrators should note, unless the history of the pull request is important, it should be integrated to a single commit using “squash and merge”. If the history is important, favor “rebase and merge”. Don’t create a merge commit unless GitHub cannot rebase the PR.
- For each major release (such as v3.0 or v3.3), an appropriately named release branch is created in the repository.
- For each minor (or “dot-dot”) release (such as v3.2.3), an appropriately named tag is created in the repository.
- In general, library changes that plausibly might break existing apps appear only in major releases, not minor ones.
## Building the engine ## Building the engine
1. Install [Gradle](http://www.gradle.org/) 1. Install [Gradle](http://www.gradle.org/)

Loading…
Cancel
Save