Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

server.eval() considered harmful by jshint #102

Open
PSzu opened this Issue Feb 20, 2014 · 2 comments

Comments

Projects
None yet
3 participants

PSzu commented Feb 20, 2014

I integrated RTD and laika with meteor boilerplate. jshint is complaining that server.eval() can be harmful... perhaps eval() should be renamed? the workaround is to use --force but still not nice...

Running "jshint:app" (jshint) task
[D] Task source: /scratch/rtd-laika-meteor-boilerplate/test/rtd/node_modules/grunt-contrib-jshint/tasks/jshint.js
Linting /scratch/rtd-laika-meteor-boilerplate/app/tests/dummyTest.js ...ERROR
[L7:C20] W061: eval can be harmful.
        server.eval(function () {

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

Aborted due to warnings.
Completed in 4.884s at Thu Feb 20 2014 21:17:30 GMT+0100 (CET) - Waiting...
Owner

arunoda commented Feb 20, 2014

Hmm.

That's a JSHint Bug. May be try evalSync.

On Fri, Feb 21, 2014 at 2:07 AM, Przemyslaw Szulczynski <
notifications@github.com> wrote:

I integrated RTD and laika with meteor bolierplate. jshint is complaining
that server.eval() can be harmful... perhaps eval() should be renamed? the
workaround is to use --force but still not nice...

Running "jshint:app" (jshint) task
[D] Task source: /scratch/rtd-laika-meteor-boilerplate/test/rtd/node_modules/grunt-contrib-jshint/tasks/jshint.js
Linting /scratch/rtd-laika-meteor-boilerplate/app/tests/dummyTest.js ...ERROR
[L7:C20] W061: eval can be harmful.
server.eval(function () {

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

Aborted due to warnings.
Completed in 4.884s at Thu Feb 20 2014 21:17:30 GMT+0100 (CET) - Waiting...


Reply to this email directly or view it on GitHubhttps://github.com/arunoda/laika/issues/102
.

Arunoda Susiripala

@arunoda http://twitter.com/arunoda
http://gplus.to/arunodahttps://github.com/arunoda
http://www.linkedin.com/in/arunoda

sbking commented Apr 5, 2014

You could try adding a .jshintrc file to your tests directory with the following content:

{
  "evil": true
}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment