Allow ignoring specific warnings.

[jordonbiondo]

From what I see there isn't a way to ignore specific warnings from js2-mode. There are certain times (usually in a test file) where I would like to ignore false warnings brought up by js2-mode.

For instance in a test file using chai's expect. I may have a line like this:

expect(true && false).to.be.false;

These lines are always highlighted because js2-mode thinks the line has no side effects even though they actually do.

My suggestion would be to have a buffer-local variable that contains a list of warnings to ignore. And then in js2-report-warning check if the warning is in the list before proceeding.

That way I can do something like this in my config in order to solve my problem.

(add-hook 'js2-mode-hook
                 (lambda ()
                   (when (is-a-test-file)
                     (add-to-list 'js2-var-with-a-list-of-ignored-warnings "msg.no.side.effects"))))

If this could be added to js2-mode I'd be happy to submit a pull request. But given the size of the project, there may be a better way to do this I overlooked.

[jordonbiondo]

I have this implemented in my config like this:

    (defvar-local jordon-js2-ignored-warnings nil)

    (defadvice js2-report-warning (around ignore-some-warnings activate)
      (unless (member (ad-get-arg 0) jordon-js2-ignored-warnings)
        ad-do-it))

Then in a hook I add certain warnings to the list depending on the type of file.

[dgutov]

Another option is to add a function to js2-post-parse-callbacks which would filter out undesired elements from js2-parsed-errors and js2-parsed-warnings.

[decoursin]

bump

[RockyRoad29]

Some details about possible solutions would be welcome on emacswiki or elsewhere with a pointer here ...
I am also trying to get rid of those annoying warnings on statements like:
http://chaijs.com/api/bdd/#method_true
Thanks