summaryrefslogtreecommitdiff
path: root/docs/internals
diff options
context:
space:
mode:
authorTim Graham <timograham@gmail.com>2017-06-12 15:39:09 -0400
committerGitHub <noreply@github.com>2017-06-12 15:39:09 -0400
commitc68f5d83c0a4ea4ccf87c7d1d5dd1e9f7b2907a3 (patch)
treee89a637c560a90c46056fe84fceece55772a23cf /docs/internals
parent3d0a0ecd2cc5d7bfe0b53c33255c1b6564c160a4 (diff)
Fixed #27655 -- Added some guidelines to the coding style docs.
Diffstat (limited to 'docs/internals')
-rw-r--r--docs/internals/contributing/writing-code/coding-style.txt21
1 files changed, 21 insertions, 0 deletions
diff --git a/docs/internals/contributing/writing-code/coding-style.txt b/docs/internals/contributing/writing-code/coding-style.txt
index e3ce27a99c..76b4918d09 100644
--- a/docs/internals/contributing/writing-code/coding-style.txt
+++ b/docs/internals/contributing/writing-code/coding-style.txt
@@ -33,6 +33,27 @@ Python style
* Use four spaces for indentation.
+* Use four space hanging indentation rather than vertical alignment::
+
+ raise AttributeError(
+ 'Here is a multine error message '
+ 'shortened for clarity.'
+ )
+
+ Instead of::
+
+ raise AttributeError('Here is a multine error message '
+ 'shortened for clarity.')
+
+ This makes better use of space and avoids having to realign strings if the
+ length of the first line changes.
+
+* Use single quotes for strings, or a double quote if the the string contains a
+ single quote. Don't waste time doing unrelated refactoring of existing code
+ to conform to this style.
+
+* Avoid use of "we" in comments, e.g. "Loop over" rather than "We loop over".
+
* Use underscores, not camelCase, for variable, function and method names
(i.e. ``poll.get_unique_voters()``, not ``poll.getUniqueVoters()``).