Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: pages/coding-style.html

Issue 29375722: Noissue - Update the JavaScript coding style guide regarding ESLint (Closed)
Patch Set: Created Feb. 15, 2017, 5:14 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pages/coding-style.html
diff --git a/pages/coding-style.html b/pages/coding-style.html
index 030223c36312fca76d2db5827140905cf10fd5fd..2d0d8d1735a8812f8dedad03d7c88f591973ccc1 100644
--- a/pages/coding-style.html
+++ b/pages/coding-style.html
@@ -26,7 +26,7 @@ title=Coding Style
<li>{{s15 Follow the Mozilla Coding Style's <a href="https://developer.mozilla.org/en-US/docs/Developer_Guide/Coding_Style#JavaScript_practices">JavaScript practices</a>.}}</li>
<li>{{javascript-strict Always use <a href="https://developer.mozilla.org/docs/Web/JavaScript/Reference/Strict_mode">strict mode</a>.}}</li>
<li>{{javascript-opening-braces Opening braces of object literals don't go on their own line when that would cause a syntax error.}}</li>
- <li>{{javascript-if-else-braces When an <code><fix>if</fix></code> statement, an <code><fix>else</fix></code> statement or a loop spans over more than one line always enclose it with braces. When an <code><fix>if</fix></code> or <code><fix>else</fix></code> statement uses braces the opposing block should do too.}}</li>
+ <li>{{javascript-if-else-braces When an <code><fix>if</fix></code> statement, an <code><fix>else</fix></code> statement or a loop spans over more than one line always enclose it with braces.}}</li>
<li>{{s18 In classes, prefix private functions with a single underscore to make them pseudo-private.}}</li>
<h3 id="javascript-modern">{{javascript-es6 Modern JavaScript (where supported)}}</h3>
<ul>
Sebastian Noack 2017/02/15 10:43:06 We probably also should add a point to clarify tha
Sebastian Noack 2017/02/15 10:43:06 While on it, mind adding a point to refer to eslin
kzar 2017/02/15 11:22:36 Done.
kzar 2017/02/15 11:22:36 Done.
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld