damian / jshint

Making it easy to lint your JavaScript assets in any Rails 3.1+ and Rails 4 application.

Home Page:https://rubygems.org/gems/jshint

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Add support for jshint ignore directive

anselmbradford opened this issue · comments

commented

Using the jshint ignore directive, e.g.:

ignoreThis(); // jshint ignore:line

(As shown in the jshint docs)

Leads to the error Bad option: 'ignore'.

Ignore certain lines is useful as some lines won't ever be updated to be compliant for a variety of reasons.

It appears you've uncovered a bug, thanks for bringing this to my attention @anselmbradford.

I'll look in to resolving this issue sometime in the next few days(it being Christmas and all), though if you can't wait, feel free to submit a PR 😄

Hi @anselmbradford - You'll be happy to know that I've just published v1.2.0 of this gem. This release includes the latest version of JSHint, which resolves your issue :)

commented

@damian Awesome! I'll check it out. Thanks much. Happy holidays.