- Committer: Brad Crittenden
- Date: 2012-09-09 19:57:35 UTC
- mfrom: (82.4.16 jshint)
- Revision ID: bac@canonical.com-20120909195735-y8ssoac8232mbmuk
Update mocha.js, wire up jshint, fix lint.
A problem with the old version of mocha.js caused the tests to fail in
Firefox. The version in test/assets was manually updated and the
version bumped in package.json.
A 'lint' target was added to the Makefile. All of the reported lint
was either fixed or silenced.
Many of the files were changed to remove tabs, one thing jshint complains about. Trailing whitespace was also deleted.
jshint complained about the following idiom:
a.foo && a.foo() // ensure the property foo exists before referencing it.
There were only two occurrences so they were fixed by converting to use 'if'.
R=gary.poster, benjamin.saller
CC=
https://codereview.appspot.com/6488096
A problem with the old version of mocha.js caused the tests to fail in
Firefox. The version in test/assets was manually updated and the
version bumped in package.json.
A 'lint' target was added to the Makefile. All of the reported lint
was either fixed or silenced.
Many of the files were changed to remove tabs, one thing jshint complains about. Trailing whitespace was also deleted.
jshint complained about the following idiom:
a.foo && a.foo() // ensure the property foo exists before referencing it.
There were only two occurrences so they were fixed by converting to use 'if'.
R=gary.poster, benjamin.saller
CC=
https://codereview.appspot.com/6488096
| Filename | Latest Rev | Last Changed | Committer | Comment | Size | ||
|---|---|---|---|---|---|---|---|
 .. |
|||||||
app
|
6 | 12 years ago | kapil.foss at gmail | reshape |
|
||
|
bin
|
1 | 12 years ago | kapil.foss at gmail | initial layout |
|
||
|
docs
|
1 | 12 years ago | kapil.foss at gmail | initial layout |
|
||
|
lib
|
43 | 12 years ago | Benjamin Saller | test aggrigation works, the matching pattern is *. |
|
||
|
test
|
12 | 12 years ago | Benjamin Saller | Includings basic router A simple test setup using |
|
||
.bzrignore |
82.4.4 | 12 years ago | Brad Crittenden | Cleaned up lint. Tests still pass. | 188 bytes |
|
|
|
config.js |
82.1.2 | 12 years ago | Benjamin Saller | review feedback | 256 bytes |
|
|
|
jshint.config |
82.4.15 | 12 years ago | Brad Crittenden | Removed mocha from test assets. Will replace with | 134 bytes |
|
|
|
Makefile |
82.4.14 | 12 years ago | Brad Crittenden | Merge and fix conflicts | 491 bytes |
|
|
|
manifest.json |
82.1.2 | 12 years ago | Benjamin Saller | review feedback | 377 bytes |
|
|
|
package.json |
82.4.12 | 12 years ago | Brad Crittenden | Merge from trunk and fix conflicts | 579 bytes |
|
|
|
README |
82.2.2 | 12 years ago | Gary Poster | respond to review | 1.8 KB |
|
|
server.js |
82.4.4 | 12 years ago | Brad Crittenden | Cleaned up lint. Tests still pass. | 268 bytes |
|
|