Skip to content
This repository has been archived by the owner on Sep 17, 2018. It is now read-only.

Deal with new JSLint options #33

Closed
Sannis opened this issue Apr 19, 2012 · 3 comments
Closed

Deal with new JSLint options #33

Sannis opened this issue Apr 19, 2012 · 3 comments

Comments

@Sannis
Copy link
Collaborator

Sannis commented Apr 19, 2012

There is some changes in JSLint defaults. I updated some of them as in @clebert fork (Sannis@7b033c5).

But there is some options, that I've revert and what to discuss:

  • regexp: false, that disallow . in regexps
  • sloppy: true, that need 'use strict' for all files
  • stupid: true, that disallow ...Sync Node.js methods

There is no problem to fix this in nodelint sources, but what about users that will be affected by such update? I need some help to decide.

// cc: @tav, @mkitt, @cliffano, @paularmstrong, @codenothing, @mranney, @azatoth

@codenothing
Copy link
Contributor

I'm thinking all need to be set to false and allow end user to change.

@paularmstrong
Copy link
Contributor

sloppy is actually a tough one to support, since it (if I remember correctly) requires you to wrap every file in a method and place use strict at the top of that method.

Output from jslint.com is Problem at line 1 character 1: Use the function form of 'use strict'.

That will affect every file that everyone has written. And is really crappy in the node.js world, since the common pattern is to not wrap everything in a function form.

regexp seems fine to support, as it's one of those things that you should only turn off if you know what you're doing.

I can go either way on stupid.

@Sannis
Copy link
Collaborator Author

Sannis commented Apr 24, 2012

Thanks!

@paularmstrong, I've create an issue in JSLint repo to discuss this: jslint-org/jslint#124

@Sannis Sannis closed this as completed Jul 3, 2012
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants