Commit Graph

220 Commits

Author SHA1 Message Date
Stefan Thomas
fee185379a Add Ripple to list of projects using this style guide. 2014-03-17 13:55:51 -07:00
Harrison Shoff
e222797041 Merge pull request #149 from fetch/simple-anchors
Make use of GitHub's default anchors.
2014-03-17 09:41:51 -07:00
Koen Punt
fb86fc032d TOC links updated 2014-03-16 16:10:41 +01:00
Koen Punt
c3a9c7f741 Rely on github's default heading anchors 2014-03-16 15:37:48 +01:00
Harrison Shoff
8bf39432c6 Merge pull request #148 from DigitPaint/add_digitpaint_to_organizations_list
Add Digitpaint to list of organisations using this style guide
2014-03-13 10:05:05 -07:00
Flurin Egger
c4d7cc0a21 Add Digitpaint to list of organisations using this style guide 2014-03-13 11:45:50 +01:00
Harrison Shoff
8c01bc4aa8 [resources] add link to es6 features 2014-03-04 09:23:44 -08:00
Harrison Shoff
5933651af1 Merge pull request #146 from moneyadviceservice/add_money_advice_service_to_readme
Add Money Advice Service to list of organisations using this style guide
2014-02-25 14:24:31 -08:00
Gareth Visagie
6c25eddf0e Add Money Advice Service to list of organisations using this style guide 2014-02-25 11:30:46 +00:00
Harrison Shoff
92a6ce613c Merge pull request #142 from gdi2290/patch-1
update copyright year
2014-02-04 09:55:56 -08:00
PatrickJS
f2e49d131a update copyright year 2014-02-03 22:38:45 -08:00
Harrison Shoff
ed3e46e498 [resources] add you might not need jquery authors 2014-01-30 11:46:02 -08:00
Harrison Shoff
7d470d860f [resources] add you might not need jquery 2014-01-30 11:44:42 -08:00
Harrison Shoff
31393c25f8 Merge pull request #139 from Peerby/master
added Peerby to list of organizations that use this guide
2014-01-22 10:26:29 -08:00
godspeedelbow
bd3914c19a added Peerby to list of organizations that use this guide 2014-01-22 13:55:54 +01:00
Harrison Shoff
6e40437c23 [readme] remove GA in favor of Github analytics 2014-01-07 10:47:36 -08:00
Harrison Shoff
d27bcfa471 [readme] add GA 2014-01-05 11:19:25 -08:00
Harrison Shoff
a3e1534faf Merge pull request #134 from airbnb/add-jshintrc
Add .jshintrc
2014-01-02 11:20:46 -08:00
Spike Brehm
b65dd90ea5 Add .jshintrc
This adds a `.jshintrc` to the `linters/` directory. I just copied over
the settings from the `SublimeLinter` file, de-nesting the JSON object
by one level.

It means we'll have to maintain both files, but it's good to have a
basic `.jshintrc` for those of us that aren't using JSHint through
Sublime Text.
2013-12-26 19:00:21 -08:00
Spike Brehm
5d99903015 Merge pull request #95 from lxe/master
Format object array to conform to whitespace guidelines
2013-12-26 18:31:36 -08:00
Harrison Shoff
425d5c7a98 Merge pull request #132 from vahan-hartooni/patch-2
Enforce 'use strict'
2013-12-17 14:33:16 -08:00
vahan-hartooni
d5db0a2f45 Enforce 'use strict'
As stated in the [Modules section](https://github.com/airbnb/javascript#modules), `'use strict'` needs to be declared at the top of a module

[JSHint has an option](http://www.jshint.com/docs/options/#strict) for something like this by enforcing the declaration of the statement at the top function scope.
2013-12-17 13:07:59 -08:00
Harrison Shoff
440b34a0b5 [resources] add link to Third Party JavaScript. fixes #128 2013-12-13 11:06:04 -08:00
Harrison Shoff
7a9f9c57f2 Merge pull request #131 from ulyssecarion/missing_hide
Add a missing `.hide()` for consistency's sake.
2013-12-13 10:55:48 -08:00
Ulysse Carion
fbc79a3bfe Add a missing .hide() for consistency's sake. 2013-12-12 19:00:45 -08:00
Harrison Shoff
c2b16dc1b2 Merge pull request #127 from airbnb/operator_whitespace
Spaces Around Operators
2013-12-05 19:54:16 -08:00
Nirav Sanghani
9469e94a07 Add guideline for spaces around operators 2013-12-05 16:35:21 -08:00
Harrison Shoff
745cf854d1 [translation] update russian fixes #120 2013-12-03 10:21:54 -08:00
Harrison Shoff
e82c8dbf56 Merge pull request #125 from vahan-hartooni/patch-1
Update SublimeLinter.sublime-settings to include maxlen enforcement
2013-12-03 10:16:58 -08:00
vahan-hartooni
898b46dc15 Update SublimeLinter.sublime-settings
The Airbnb guide states: "Strings longer than 80 characters should be written across multiple lines using string concatenation."

I propose to add the maxlen option to enforce this rule: http://www.jshint.com/docs/options/#maxlen

However maxlen enforces the entire source code, not just a line of String. I would like to hear your thoughts on whether that is a beneficial enforcement to keep JS code at a set width.
2013-12-02 10:46:34 -08:00
Harrison Shoff
3a6aff1f13 [in the wild] add REI 2013-11-26 19:15:17 -08:00
Harrison Shoff
e8764fb20b Merge pull request #122 from bmorrall/patch-1
Concatenation examples are now equivalent
2013-11-25 21:43:37 -08:00
Ben Morrall
d44c4e01ea Concatenation examples are now equivalent 2013-11-26 15:17:25 +11:00
Harrison Shoff
f07de4befb [translation] add Bulgarian translation. fixes #121 2013-11-24 21:35:53 -08:00
Harrison Shoff
1ffec40ffd Merge pull request #119 from sbezludny/patch-1
Added Russian translation
2013-11-21 14:31:51 -08:00
Serg
8702dd4b83 Added Russian translation 2013-11-22 00:01:50 +02:00
Harrison Shoff
30957f208a Merge pull request #117 from nmussy/french_translation
Added French translation to the list
2013-11-19 10:41:42 -08:00
Jimmy Gaussen
30d8ea098e copy pasta 2013-11-19 19:40:06 +01:00
Jimmy Gaussen
c29818caa3 Added French translation to the list 2013-11-19 19:31:10 +01:00
Harrison Shoff
6dff6bc313 Merge pull request #116 from AeonAxan/patch-1
Remove invalid name - Naming Conventions
2013-11-18 07:39:14 -08:00
Ahmed Aeon Axan
6445a6fd00 Remove invalid name - Naming Conventions
In naming conventions `var this-is-my-object = {};` is given as an example of a bad name. But since this is an invalid name in Javascript. I dont think it is needed since its a syntax error.
2013-11-17 21:07:54 +05:30
Harrison Shoff
aaaf392e1c [type casting & coercion] fix formatting 2013-11-14 14:05:28 -08:00
Harrison Shoff
8bab767301 [type casting & coercion] add note about bitshift and large integers. fixes #109 2013-11-14 14:02:01 -08:00
Harrison Shoff
fa32d9a158 [in the wild] add Gawker Media 2013-11-13 10:52:26 -08:00
Harrison Shoff
ac5827d9e4 Merge pull request #108 from tipjs/master
Add Korean version
2013-11-13 10:11:07 -08:00
SeungHyun PAEK
775ba4e2e9 Add Korean version 2013-11-13 14:03:15 +09:00
Harrison Shoff
49d480b22d Merge pull request #106 from AanZee/master
added Aan Zee to 'In the Wild'
2013-11-12 17:27:28 -08:00
Jelle Kralt
e8caad29d4 Update README.md 2013-11-12 17:26:13 +01:00
Harrison Shoff
c7ab9b8cec Merge pull request #103 from pborreli/patch-1
Fixed typo
2013-11-10 17:13:35 -08:00
Pascal Borreli
de4979b999 Fixed typos 2013-11-10 23:27:33 +00:00