Skip to content

Commit

Permalink
fixed some sentence construction
Browse files Browse the repository at this point in the history
  • Loading branch information
A-UNDERSCORE-D committed Sep 9, 2020
1 parent 009ded5 commit ee12b20
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions Contributing.md
Original file line number Diff line number Diff line change
Expand Up @@ -194,9 +194,9 @@ and another for the Pull Request, merging or cherry-picking commits as needed.

Yes, this means using 'color' rather than 'colour', and in general will mean US, not British, spellings.

## **ALWAYS** place a single-statement control flow body
## Control flow

e.g." control statements such as `if`, `else`, `for`, `foreach`, on a separate line, with consistent indentation.
Never oneline any control flow (`if`, `else`, `for`), as it makes spotting what happens next difficult.

Yes:

Expand All @@ -213,7 +213,9 @@ if something_true: one_thing_we_do()

Yes, some existing code still flouts this rule.

### **Always** use Line breaks after scope changes. It makes reading code far easier
### Scope changes

**Always** use Line breaks after scope changes. It makes seeing where scope has changed far easier on a quick skim

Yes:

Expand Down

0 comments on commit ee12b20

Please sign in to comment.