Skip to content

Commit

Permalink
Issue felixge#15: Verbose/Complete Local Requirements
Browse files Browse the repository at this point in the history
  • Loading branch information
GPHofficial committed Oct 22, 2015
1 parent eeb59ca commit 049d7ff
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions Readme.md
Original file line number Diff line number Diff line change
Expand Up @@ -40,9 +40,11 @@ according to your preferences.
* [No nested closures](#no-nested-closures)
* [Use slashes for comments](#use-slashes-for-comments)
* [Object.freeze, Object.preventExtensions, Object.seal, with, eval](#objectfreeze-objectpreventextensions-objectseal-with-eval)
* [Requires At Top](#Requires At Top)
* [Getters and setters](#getters-and-setters)



## 2 Spaces for indention

Use 2 spaces for indenting your code and swear an oath to never mix tabs and
Expand Down Expand Up @@ -524,6 +526,10 @@ if (isSessionValid) {

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

## Requires At Top

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.

## Getters and setters

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

0 comments on commit 049d7ff

Please sign in to comment.