github/javascript
1
1
Fork 0
mirror of https://github.com/airbnb/javascript.git synced 2026-01-14 02:28:20 -05:00
Code Issues Packages Projects Releases Wiki Activity
206 Commits 5 Branches 102 Tags
bd3914c19a86e48284145239ea749f209c061c25
Commit Graph

206 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
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
Harrison Shoff
9862309820 [in the wild] add Mighty Spring 2013-11-09 12:03:02 -08:00
Harrison Shoff
ed6df64332 [resources] add link to popular js conventions on github 2013-11-08 12:03:21 -08:00
Harrison Shoff
f25396e07e Merge pull request #99 from ultimatedelman/master 
[functions] define var in example at the top
 2013-11-08 11:20:17 -08:00
Jason Edelman
cce1a2d3e0 Update README.md 
always define your variables at the top. defining them elsewhere is confusing because of hoisting. there is no block scope in javascript.
 2013-11-08 11:06:43 -08:00
Aleksey Smolenchuk
65a86bc4a6 Format object array to conform to whitespace guidelines 2013-11-07 17:23:47 -05:00
Harrison Shoff
95b6b5f6c9 Merge pull request #93 from paolocarrasco/patch-1 
Adding a Spanish version to the list of translations
 2013-10-31 16:30:48 -07:00
Paolo Carrasco
16acfdad65 Adding a Spanish version to the list of translations 2013-10-31 20:42:31 -02:00
Harrison Shoff
9d17310d29 [resources] add additional helpful resources. fixes #92 2013-10-31 12:08:25 -07:00
Harrison Shoff
fcdf61c9ec Merge pull request #91 from jdhartley/patch-1 
Fix jQuery selector with context param order
 2013-10-28 13:58:20 -07:00
JD Hartley
e84b63c07f Fix jQuery selector with context param order 
`$(selector, parent)`

See: http://api.jquery.com/jQuery/#jQuery-selector-context
 2013-10-28 14:15:37 -05:00
Harrison Shoff
6394ca8dd8 [jquery] the faster lookup isn't actually faster. Fixes #90 2013-10-25 14:20:42 -07:00
Harrison Shoff
5a769c1c0e [resources] add superhero.js 2013-10-24 13:37:02 -07:00
Harrison Shoff
c4a44b5753 [resources] add human javascript 2013-10-21 09:57:17 -07:00
Harrison Shoff
6cc601729f Merge pull request #89 from AnSavvides/master 
Add Secrets of the JavaScript Ninja to list of books
 2013-10-21 09:40:42 -07:00