summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorCarlton Gibson <carlton.gibson@noumenal.es>2020-07-21 08:45:09 +0200
committerCarlton Gibson <carlton.gibson@noumenal.es>2020-10-15 10:10:49 +0200
commit848e48c62dfc097e1cc6deec240336f657bb491c (patch)
treedf1273279fd3bc8a86665f377ebafad3a1c69fb6
parent536213278c77884e9f985c18a8f9e1158f0d680a (diff)
[3.1.x] Fixed #29988 -- Updated coding style to allow f-strings.
Thanks to Nick Pope for review. Backport of 411cc0ae18aa9c64447eed229e65e092d23f80e6 from master
-rw-r--r--docs/internals/contributing/writing-code/coding-style.txt31
1 files changed, 31 insertions, 0 deletions
diff --git a/docs/internals/contributing/writing-code/coding-style.txt b/docs/internals/contributing/writing-code/coding-style.txt
index 56f6bd5738..dcbef85bb7 100644
--- a/docs/internals/contributing/writing-code/coding-style.txt
+++ b/docs/internals/contributing/writing-code/coding-style.txt
@@ -52,6 +52,37 @@ Python style
single quote. Don't waste time doing unrelated refactoring of existing code
to conform to this style.
+* String variable interpolation may use
+ :py:ref:`%-formatting <old-string-formatting>`, :py:ref:`f-strings
+ <f-strings>`, or :py:meth:`str.format` as appropriate, with the goal of
+ maximizing code readability.
+
+ Final judgments of readability are left to the Merger's discretion. As a
+ guide, f-strings should use only plain variable and property access, with
+ prior local variable assignment for more complex cases::
+
+ # Allowed
+ f'hello {user}'
+ f'hello {user.name}'
+ f'hello {self.user.name}'
+
+ # Disallowed
+ f'hello {get_user()}'
+ f'you are {user.age * 365.25} days old'
+
+ # Allowed with local variable assignment
+ user = get_user()
+ f'hello {user}'
+ user_days_old = user.age * 365.25
+ f'you are {user_days_old} days old'
+
+ f-strings should not be used for any string that may require translation,
+ including error and logging messages. In general ``format()`` is more
+ verbose, so the other formatting methods are preferred.
+
+ Don't waste time doing unrelated refactoring of existing code to adjust the
+ formatting method.
+
* 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