Skip to content

Commit 3d4db19

Browse files
committed
Merge pull request felixge#78 from GPHofficial/GPHofficial-patch-1
Patch for felixge#58 and felixge#15
2 parents f0eec92 + dabf85e commit 3d4db19

File tree

1 file changed

+6
-4
lines changed

1 file changed

+6
-4
lines changed

Readme.md

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -52,11 +52,13 @@ according to your preferences.
5252

5353
### Miscellaneous
5454
* [Object.freeze, Object.preventExtensions, Object.seal, with, eval](#objectfreeze-objectpreventextensions-objectseal-with-eval)
55+
* [Requires At Top](#requires-at-top)
5556
* [Getters and setters](#getters-and-setters)
5657
* [Do not extend built-in prototypes](#do-not-extend-built-in-prototypes)
5758

5859
## Formatting
5960

61+
6062
### 2 Spaces for indention
6163

6264
Use 2 spaces for indenting your code and swear an oath to never mix tabs and
@@ -207,10 +209,6 @@ function bank_Account() {
207209
Constants should be declared as regular variables or static class properties,
208210
using all uppercase letters.
209211

210-
Node.js / V8 actually supports mozilla's [const][const] extension, but
211-
unfortunately that cannot be applied to class members, nor is it part of any
212-
ECMA standard.
213-
214212
*Right:*
215213

216214
```js
@@ -528,6 +526,10 @@ if (isSessionValid) {
528526

529527
Crazy shit that you will probably never need. Stay away from it.
530528

529+
### Requires At Top
530+
531+
Always put requires at top of file to clearly illustrate a file's dependencies. Besides giving an overview for others at a quick glance of dependencies and possible memory impact, it allows one to determine if they need a package.json file should they choose to use the file elsewhere.
532+
531533
### Getters and setters
532534

533535
Do not use setters, they cause more problems for people who try to use your

0 commit comments

Comments
 (0)