Skip to content

Commit

Permalink
Merge pull request #2983 from solidusio/jacobherrington-patch-1
Browse files Browse the repository at this point in the history
Add criteria for merging a PR
  • Loading branch information
jacobherrington authored Dec 13, 2018
2 parents e2ef677 + aff6800 commit cb8c67a
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 @@ -46,6 +46,14 @@ At this point you're waiting on us. We like to at least comment on, if not
accept, pull requests within three business days (and, typically, one business
day). We may suggest some changes or improvements or alternatives.

Typically, Pull Requests can be merged when they have met the following criteria:

- Open for at least 24 hours
- Passing CI
- 2 or more Core Team approvals

Though in some cases the Core Team may choose not to merge a PR that meets these requirements.

### Pull Request guidelines

* The specs must pass for each individual commit.
Expand Down

0 comments on commit cb8c67a

Please sign in to comment.