jrcryer / generator-mean

Yeoman generator for a MEAN stack, inspired by mean.io
MIT License
120 stars 25 forks source link

yo meanstack instead of yo;jshint erros after grunt #19

Open naman opened 10 years ago

naman commented 10 years ago

After running grunt for the first time, a huge set of warnings are shown.

Linting app/scripts/app.js ...ERROR [L9:C17] W015: Expected 'templateUrl' to have an indentation at 11 instead at 17. templateUrl: 'views/main.html', Linting app/scripts/app.js ...ERROR [L10:C17] W015: Expected 'controller' to have an indentation at 11 instead at 17. controller: 'MainCtrl' Linting app/scripts/app.js ...ERROR [L11:C13] W015: Expected '}' to have an indentation at 9 instead at 13. }) Linting app/scripts/app.js ...ERROR [L13:C17] W015: Expected 'redirectTo' to have an indentation at 11 instead at 17. redirectTo: '/' Linting app/scripts/app.js ...ERROR [L14:C13] W015: Expected '}' to have an indentation at 9 instead at 13. }); Linting app/scripts/app.js ...ERROR [L15:C5] W015: Expected '}' to have an indentation at 7 instead at 5. }); Linting app/scripts/app.js ...ERROR [L17:C1] W015: Expected '}' to have an indentation at 3 instead at 1. }.call(this)); Linting app/scripts/app.js ...ERROR [L17:C8] W040: Possible strict violation. }.call(this));

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

Aborted due to warnings. [~/Desktop/yeoprac] $ grunt Running "jshint:all" (jshint) task Linting app/scripts/app.js ...ERROR [L9:C17] W015: Expected 'templateUrl' to have an indentation at 11 instead at 17. templateUrl: 'views/main.html', Linting app/scripts/app.js ...ERROR [L10:C17] W015: Expected 'controller' to have an indentation at 11 instead at 17. controller: 'MainCtrl' Linting app/scripts/app.js ...ERROR [L11:C13] W015: Expected '}' to have an indentation at 9 instead at 13. }) Linting app/scripts/app.js ...ERROR [L13:C17] W015: Expected 'redirectTo' to have an indentation at 11 instead at 17. redirectTo: '/' Linting app/scripts/app.js ...ERROR [L14:C13] W015: Expected '}' to have an indentation at 9 instead at 13. }); Linting app/scripts/app.js ...ERROR [L15:C5] W015: Expected '}' to have an indentation at 7 instead at 5. }); Linting app/scripts/app.js ...ERROR [L17:C1] W015: Expected '}' to have an indentation at 3 instead at 1. }.call(this)); Linting app/scripts/app.js ...ERROR [L17:C8] W040: Possible strict violation. }.call(this));

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

Aborted due to warnings.

jrcryer commented 10 years ago

Hi, sorry about not getting back to you sooner!

I'll take a look into this - PRs are welcomed :)

mhaseebkhan commented 10 years ago

Hi,

I'm facing same issue. Any update on the fix?

naman commented 10 years ago

Nothing yet.

jrcryer commented 10 years ago

Sorry guys - been really busy of late - will try to get something sorted. PRs are welcome!

jrcryer commented 10 years ago

Can you confirm the versions you are running + platform?

→ node --version && npm --version 
v0.10.32
2.1.4

And I get all files linted as expected. No errors.