add info about 'making pull requests'

etpinard · etpinard · commit 86cf33c1d13c · 2016-04-26T13:09:35.000-04:00
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
@@ -17,6 +17,20 @@ community.plot.ly (tagged [`plotly-js`](http://community.plot.ly/c/plotly-js)) o
 Comments on GitHub issues or pull requests should add content to the discussions. 
 Approbation comments such as *+1* or *I would like this feature to be implemented as well* will be deleted by the maintainers. Please use [GitHub reactions](https://github.com/blog/2119-add-reactions-to-pull-requests-issues-and-comments) instead.
 
+## Making pull requests
+
+Developers are strongly encouraged to first make a PR to their own plotly.js
+fork and ask one of the maintainers to review the modifications there. Once the
+pull request is deemed satisfactory, the developer will be asked to make a pull
+request to the main plotly.js repo and may be asked to squash some commits
+before doing so.
+
+It is forbidden to force push (i.e. `git push -f`) to remote branches
+associated with opened pull requests. Force pushes make it hard for maintainers 
+to keep track of updates. Therefore, if required, please `git merge
+master` into your PR branch instead of `git rebase master`.
+
+
 ## Issue labels
 
 We use the following [labels](https://github.com/plotly/plotly.js/labels) to track issues:
