From 72616e7eb9be77c4dc94ff73fde131a0be87c13f Mon Sep 17 00:00:00 2001 From: Mattes D Date: Thu, 21 Dec 2017 12:25:49 +0100 Subject: [PATCH] Added Github-related rules to Contributing.md (#4076) These should make it easier for us to review code. --- CONTRIBUTING.md | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 8270c51a8..7a1d85a05 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -76,6 +76,15 @@ src/CheckBasicStyle.lua 1>&2 -g Note that the check script is not smart enough to catch everything, so not having any warnings does not necessarily imply that you followed the conventions fully. The other humans working on this will perform more checks before merging. +Github-related conventions +-------------------------- +The following rules are not strictly enforced, but if you follow them, you will make it easier for us to review your changes. + - Use good short title for your PRs. `Store Health as float` is good, `Fix #4024` is not good. + - If a PR fixes an issue, mention it in the PR description, but **not** in the commits themselves. Typically, we add the line `Fixes #007` to the bottom of the PR description, this makes Github link the PR to the issue, auto-close the issue when the PR is merged, and also is included in the merge message. + - Focus on one thing only. Each PR should avoid making unrelated changes - those makes the history more difficult to understand later on, and they make the review a headache. + - Feel free to rebase, amend or otherwise manipulate the commits in the PR (of course, as long as you're the only one working on the PR). We mostly squash-merge PRs, so your internal commit structure will not be important in the end. + - If the PR is about one change, but it makes sense to keep the change broken into several commits, make sure each such commit is compilable, encapsulated, and possibly add a note to the PR description so that we consider this. + Copyright ---------