|
| 1 | +# Contribution Guidelines |
| 2 | + |
| 3 | +## Branches Introduction |
| 4 | + |
| 5 | +We follow a simple pattern for collaborating with multiple developers, that makes us easy to manage our codebase. Please find the description of branches below. |
| 6 | + |
| 7 | + |
| 8 | +| Branch Name | Description | |
| 9 | +| ------ | ------ | |
| 10 | +| **master** | Production branch [Never ever ever ever commit directly to this branch]. | |
| 11 | +| **staging** | Staging branch [For testing production code before pushing to production]. | |
| 12 | +| **develop** | Development branch [ Recent development changes]. | |
| 13 | +| **patches** | Small patches for any branch. | |
| 14 | +| **tasks** | Branches for Collaborators to work on. | |
| 15 | + |
| 16 | +## Maintainers |
| 17 | + |
| 18 | +#### Responsibilites |
| 19 | + - Ensure any install or build dependencies are removed before the end of the layer when doing a build. |
| 20 | + - Increase the version numbers in any examples files and the README.md to the new version that this Pull Request would represent. The versioning scheme we use is SemVer. |
| 21 | + - Delete patches branch after merging. |
| 22 | + - Delete task brached after accepting PR. |
| 23 | + - Maintainers are responsible for Integration testing. |
| 24 | + |
| 25 | +#### Do's and Don'ts |
| 26 | + - New task branch creation and assigning to the collaborators. |
| 27 | + - Accept PR only after testing and reviewing the code. |
| 28 | + - Do not commit changes direcly on any parent branch. |
| 29 | + - Do not accept PR without any unit tests. |
| 30 | + |
| 31 | +# Collaborators |
| 32 | + |
| 33 | +#### Responsibilites |
| 34 | + - Clone Repository on your local machine, and do changes in your assigned branch only. |
| 35 | + - Once the task is completed pull request on **develop branch** only, any pull request on Master or Staging will not be accepted. |
| 36 | + - Request PR review to repsitory's mantianer. If you don't do that, that will not be reviwed and merged to parent branch. |
| 37 | + |
| 38 | +#### Do's and Don'ts |
| 39 | + - Follow coding styles. |
| 40 | + - Check if your changed are consistent with the coding styles before PR. |
| 41 | + - Run unit test before PR. |
| 42 | +# Outside Conrtibutors |
| 43 | + |
| 44 | +We are not open for Outside collaborators yet, please connect on Tusharmudagal3@gmail.com if you are interested in a particular task. |
0 commit comments