From 6b988e673d639cb7ea1165aa7c6348b5a018aa53 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Deniz=20T=C3=BCrkoglu?= Date: Wed, 30 Sep 2015 08:00:22 -0700 Subject: [PATCH] Documentation Summary: Add instruction how to run tests prior to pushing PR'sCloses https://github.com/facebook/infer/pull/145 Reviewed By: @cristianoc Differential Revision: D2493665 Pulled By: @jvillard --- CONTRIBUTING.md | 21 ++++++++++++++++++++- 1 file changed, 20 insertions(+), 1 deletion(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 347e7b52b..fe7163962 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -1 +1,20 @@ -We require contributors to sign our Contributor License Agreement. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla. If you have any questions, please drop us a line at cla@fb.com. Thanks! +# Contribution Guidelines + +## Contributor License Agreement + +We require contributors to sign our Contributor License Agreement. In +order for us to review and merge your code, please sign up at +https://code.facebook.com/cla. If you have any questions, please drop +us a line at cla@fb.com. Thanks! + +## Submitting Pull Requests + +* Please make sure Infer builds: `make -C infer clean `. Refer + to the [installation + document](https://github.com/facebook/infer/blob/master/INSTALL.md) + for details. + +* If relevant, add a test for your change. Tests are located at + `infer/tests/codetoanalyze/` and + `infer/tests/{endtoend,frontend}/`. To run the all tests, execute + `./scripts/test.sh`.