From 20d9ec1fd2e2266786efc0d30bc387dc012cc7c3 Mon Sep 17 00:00:00 2001 From: Eli Uriegas Date: Fri, 14 Apr 2017 14:38:45 -0500 Subject: [PATCH] Fix the numbered list --- CONTRIBUTING.md | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 95922693..367956b6 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -30,12 +30,12 @@ See it's that simple! So the pull request approval rules are pretty simple: 1. All pull requests must pass unit tests -* All pull requests must be reviewed and approved by at least +2. All pull requests must be reviewed and approved by at least one current collaborator on the project -* All pull requests must pass flake8 checks -* If you decide to remove/change anything from any common interface +3. All pull requests must pass flake8 checks +4. If you decide to remove/change anything from any common interface a deprecation message should accompany it. -* If you implement a new feature you should have at least one unit +5. If you implement a new feature you should have at least one unit test to accompany it. ## Documentation