Skip to content

Commit

Permalink
Add information about our PR merge policy
Browse files Browse the repository at this point in the history
  • Loading branch information
AliSoftware committed Oct 27, 2020
1 parent 17d1f27 commit d313a55
Showing 1 changed file with 8 additions and 0 deletions.
8 changes: 8 additions & 0 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,14 @@ The core team monitors and reviews all pull requests. Depending on the changes,

We do our best to respond quickly to all pull requests. If you don't get a response from us after a week, feel free to reach out to us via Slack.

Note: If you are part of the org and have the permissions on the repo, don't forget to assign yourself to the PR, and add the appropriate GitHub label and Milestone for the PR

### PR merge policy

* PRs require one reviewer to approve the PR before it can be merged to the base branch
* We keep the PR git history when merging (merge via "merge commit")
* The reviewer who approved the PR will merge it right after approval (without waiting for the PR author) if all checks are green.

### Development Practices

Have look at the [Coding Style Guide](docs/coding-style.md) to learn how to format your code so it passes the convention and quality checks like the rest of the project. The [docs](docs/) contain all the development guides like [what a good pull request looks like](docs/pull-request-guidelines.md), and [how to use String and Drawable resources](docs/using-android-resources.md).
Expand Down

0 comments on commit d313a55

Please sign in to comment.