Update gulp task to lint only relevant scripts

[ci skip]

Signed-off-by: Alex Coles <alex@alexbcoles.com>
Add more comprehensive JSHint config

Most defaults based on:
https://github.com/thoughtbot/hound/blob/master/config/style_guides/javascript.json

[ci skip]

Signed-off-by: Alex Coles <alex@alexbcoles.com>
Tell JSHint we use a node environment

[ci skip]

Signed-off-by: Alex Coles <alex@alexbcoles.com>
Tell JSHint to allow jQuery globals

[ci skip]

Signed-off-by: Alex Coles <alex@alexbcoles.com>
Tell JSHint to allow various application globals

[ci skip]

Signed-off-by: Alex Coles <alex@alexbcoles.com>
Tell JSHint to allow globals for Karma/Mocha tests

[ci skip]

Signed-off-by: Alex Coles <alex@alexbcoles.com>
Tell JSHint to allow globals for Protractor tests

[ci skip]

Signed-off-by: Alex Coles <alex@alexbcoles.com>
Tell JSHint to allow longer lines (as Rubocop)

See also 442a5107.

[ci skip]

Signed-off-by: Alex Coles <alex@alexbcoles.com>
Fix semicolons in Angular app to placate JSHint

Most final semicolons missing after conversion to CommonJS modules
(see 5f6696d5).

Signed-off-by: Alex Coles <alex@alexbcoles.com>
Fix semicolons in Karma tests to placate JSHint

Signed-off-by: Alex Coles <alex@alexbcoles.com>
Prefer object literal notation in Karma tests

Signed-off-by: Alex Coles <alex@alexbcoles.com>
Quieten JSHint warnings about Chai BDD assertions

Signed-off-by: Alex Coles <alex@alexbcoles.com>
Fix semicolon in Protractor test to placate JSHint
Remove deleted trailing option from JSHint config

Functionality removed from JSHint.
See: https://github.com/jshint/jshint/pull/1554

Signed-off-by: Alex Coles <alex@alexbcoles.com>
Merge pull request #2184 from opf/feature/jshint-config-with-fixes