count.js grunt lint fixes: #71

Merged
merged 1 commit into from Jun 19, 2013

Conversation

Projects
None yet
2 participants
@tomfuertes
Contributor

tomfuertes commented Jun 19, 2013

$ git checkout master
Already on 'master'

$ grunt
Running "jshint:all" (jshint) task
Linting tests/app/count.js...ERROR
[L21:C9] Read only.
        console = {};
[L34:C39] Expected an assignment or function call and instead saw an expression.
        expect(nums.length > 1).to.be.ok;
[L35:C39] Expected an assignment or function call and instead saw an expression.
        expect(nums.length < 5).to.be.ok;
[L53:C39] Expected an assignment or function call and instead saw an expression.
        expect(nums.length < 5).to.be.ok;

Warning: Task "jshint:all" failed. Use --force to continue.

Aborted due to warnings.

$ git checkout jshints
Switched to branch 'jshints'

$ grunt
Running "jshint:all" (jshint) task
>> 26 files lint free.

Done, without errors.
count.js grunt lint fixes:
* expr:true
* global console:true

rmurphey added a commit that referenced this pull request Jun 19, 2013

Merge pull request #71 from tomfuertes/jshints
count.js grunt lint fixes:

@rmurphey rmurphey merged commit 811e134 into rmurphey:master Jun 19, 2013

@tomfuertes tomfuertes deleted the tomfuertes:jshints branch Feb 20, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment