Merge pull request #55 from mapado/add-review-rules

add review rules

Author: potlaetitia

README.md

@@ -20,6 +20,8 @@ $ yarn
 $ yarn start
 ```
 
+Open URL in your browser `http://xxx.dev.mapado.com:3001/best-practices/`
+
 This command starts a local development server and open up a browser window. Most changes are reflected live without having to restart the server.
 
 ### Build

docs/common/pull-request.md

@@ -8,3 +8,16 @@ Une pull request doit respecter les règles suivantes avant d'être mergée :
 - Favoriser les PR de - de 1000 lignes (avec tests) / 600 lignes de code pur
 
 NB : en cas d'absence de review, c'est à l'auteur de la PR de relancer. Si l'équipe concernée ne review pas la PR rapidement, le remonter au point tech en rappelant la responsabilité collective de la review.
+
+### Statuts des reviews
+
+#### Approved
+La PR est validée, la relecture peut contenir des commentaires et suggestions qui PEUVENT être pris en compte.
+
+#### Request Changes
+La PR est rejetée. Le rejet comporte des commentaires qui DOIVENT être pris en compte.
+
+Une fois la PR corrigée, l'auteur DOIT "re-request" les reviewers. Si une PR comporte au moins 1 rejet, elle NE DOIT PAS être mergée en l'état.
+
+#### Comment
+PEUT exister dans le cas de question qui mérite des échanges, mais dans l'idéal NE DEVRAIT PAS exister. Chaque PR relue DEVRAIT être validée ou rejetée.