using javascript-tide before javascript-eslint #422

Aaronzinhoo · 2021-03-02T17:41:49Z

Aaronzinhoo
Mar 2, 2021

Hey, so I have been using rjsx-mode for some time now and since I recently learned of the tide checkers I wanted to add javascript tide to its checkers. I was able to successfully load it, but every time it loads, javascript-eslint is loaded before it. Is there any way to change the ordering so that javascript-tide is preferred or loaded before javascript-eslint?

:config
  (flycheck-add-mode 'javascript-eslint 'rjsx-mode)
  (flycheck-add-next-checker 'jsx-tide '(warning . javascript-eslint) 'append)
  (flycheck-add-next-checker 'javascript-tide '(warning . javascript-eslint) 'append)
  (flycheck-add-mode 'typescript-tide 'ng2-ts-mode)
  (flycheck-add-mode 'javascript-eslint 'ng2-ts-mode)
  (flycheck-add-mode 'typescript-tslint 'ng2-ts-mode)
  (flycheck-add-next-checker 'typescript-tide '(warning . javascript-eslint) 'append)
  (if (file-exists-p (concat tide-project-root "node_modules/typescript/bin/tsserver"))
      (setq tide-tsserver-executable "node_modules/typescript/bin/tsserver"))

This is the relevant portion of my config. I also modified the javascript-tide modes it activates for. Any suggesitons?

I got it to work by creating a hook function and disabling the add-next-checker line. My tide-mode hook simply starts the tide server and sets some presets I like for my dev environment. I do not think this is a good solution but it works at least. If you know a better method I would be open to suggestions.

(defun aaronzinhoo-tide-rjsx-mode-hook ()
    (aaronzinhoo-tide-mode-hook)
    (flycheck-select-checker 'javascript-tide))

ananthakumaran · 2021-03-11T10:06:38Z

ananthakumaran
Mar 11, 2021
Maintainer

I don't see an officially documented approach to set the next checkers flycheck/flycheck#836 flycheck/flycheck#1660

The way you do currently seems ok to me. One thing to note is, we define multiple tide checkers {mode-name}-tide. For rjsx mode, jsx-tide should be used.

// after flycheck and tide is loaded
(flycheck-add-next-checker 'jsx-tide '(warning . javascript-eslint))

// in your rjsx mode hook
(setq flycheck-checker 'jsx-tide)

0 replies

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

using javascript-tide before javascript-eslint #422

Uh oh!

{{title}}

Uh oh!

Uh oh!

{{editor}}'s edit

{{editor}}'s edit

Uh oh!

Replies: 1 comment

Uh oh!

{{title}}

Uh oh!

Select a reply

Uh oh!

using javascript-tide before javascript-eslint #422

Uh oh!

Uh oh!

Aaronzinhoo Mar 2, 2021

Replies: 1 comment

Uh oh!

ananthakumaran Mar 11, 2021 Maintainer

Aaronzinhoo
Mar 2, 2021

ananthakumaran
Mar 11, 2021
Maintainer