As per title.
deadalnix jasonbcox mbhinder jimpo matiu Mengerian
- Group Reviewers
- rSTAGING81c8fef18aeb: Update developer notes and link to it from contributing docs
rABC81c8fef18aeb: Update developer notes and link to it from contributing docs
Read the documentatino
|28 ↗||(On Diff #3550)|
Curly braces should be on the same line as the definition. Same goes for lines below this one.
|53 ↗||(On Diff #3550)|
Rather than what this code does, perhaps "Comment summarizing the intended purpose of this section of code". Unless the code is particularly difficult to parse, describing what it does is a bit redundant. The intended purpose is often very similar to what it does, but we all know that isn't always the case. :P
|324 ↗||(On Diff #3550)|
Should we leave this in until T214 is completed?
|18 ↗||(On Diff #3550)|
Remove the allowance for single-line if statements.
|39 ↗||(On Diff #3550)|
Presumably lowerCamelCase for local variables too? Can we document that somewhere?
|44 ↗||(On Diff #3550)|
Can you make it explicit that this applies to both functions and class methods?
|56 ↗||(On Diff #3550)|