diff options
author | Aleksey Kladov <[email protected]> | 2021-05-10 17:04:41 +0100 |
---|---|---|
committer | Aleksey Kladov <[email protected]> | 2021-05-14 14:19:27 +0100 |
commit | 0650f77dd9defaf352f81c5ee4ee73a1eae942b7 (patch) | |
tree | 4758b6ea999292cf18a078dd942f51621a2d68ea /docs/dev | |
parent | ab528e85f71da188722ae031b31a3a70bac1cadd (diff) |
internal: remove one more immutable tree
Diffstat (limited to 'docs/dev')
-rw-r--r-- | docs/dev/style.md | 22 |
1 files changed, 22 insertions, 0 deletions
diff --git a/docs/dev/style.md b/docs/dev/style.md index 9b0ddec66..f22b69768 100644 --- a/docs/dev/style.md +++ b/docs/dev/style.md | |||
@@ -951,6 +951,28 @@ match p.current() { | |||
951 | 951 | ||
952 | ## Documentation | 952 | ## Documentation |
953 | 953 | ||
954 | Style inline code comments as proper sentences. | ||
955 | Start with a capital letter, end with a dot. | ||
956 | |||
957 | ```rust | ||
958 | // GOOD | ||
959 | |||
960 | // Only simple single segment paths are allowed. | ||
961 | MergeBehavior::Last => { | ||
962 | tree.use_tree_list().is_none() && tree.path().map(path_len) <= Some(1) | ||
963 | } | ||
964 | |||
965 | // BAD | ||
966 | |||
967 | // only simple single segment paths are allowed | ||
968 | MergeBehavior::Last => { | ||
969 | tree.use_tree_list().is_none() && tree.path().map(path_len) <= Some(1) | ||
970 | } | ||
971 | ``` | ||
972 | |||
973 | **Rationale:** writing a sentence (or maybe even a paragraph) rather just "a comment" creates a more appropriate frame of mind. | ||
974 | It tricks you into writing down more of the context you keep in your head while coding. | ||
975 | |||
954 | For `.md` and `.adoc` files, prefer a sentence-per-line format, don't wrap lines. | 976 | For `.md` and `.adoc` files, prefer a sentence-per-line format, don't wrap lines. |
955 | If the line is too long, you want to split the sentence in two :-) | 977 | If the line is too long, you want to split the sentence in two :-) |
956 | 978 | ||