From 09e57d228c75dec9699ce5f45c8b1a84fe13a0bb Mon Sep 17 00:00:00 2001 From: Nikos Mavrogiannopoulos Date: Tue, 31 Oct 2017 09:18:15 +0100 Subject: CONTRIBUTING.md: added a short text on reviewing code [ci skip] Signed-off-by: Nikos Mavrogiannopoulos --- CONTRIBUTING.md | 21 +++++++++++++++++++++ 1 file changed, 21 insertions(+) (limited to 'CONTRIBUTING.md') diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index bec4f00284..7947f7e6d2 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -267,3 +267,24 @@ two things must be done: Note that, for constants and enums, "schemefied" names are used, as noted under the "Guile API Conventions" node of the manual. + + +# Reviewing code + +A review as part of the gitlab merge requests, is a way to prevent errors due to +these guidelines not being followed, e.g., verify there is a reasonable test suite, +and whether it covers reasonably the new code, that the function naming is +consistent with these guidelines, as well as check for obvious mistakes in the new +code. + +The intention is to keep reviews lightweight, and rely on CI for tasks such +as compiling and testing code and features. + +A proposed checklist to assist such reviews follows. + * [ ] There is a test suite reasonably covering new functionality or modifications + * [ ] Function naming, parameters, return values, types, etc., are consistent and according to `CONTRIBUTION.md` + * [ ] This feature/change has adequate documentation added + * [ ] No obvious mistakes in the code + + +[Guidelines to consider when reviewing.](https://github.com/thoughtbot/guides/tree/master/code-review) -- cgit v1.2.1