Почему eslint ругается no-unused-vars?

Question

[Валерий Чупурнов]

Доброго дня. Использую webpack + babel + eslint
Есть файл Helpers.js
В нем такого рода функции(украдены из jQuery):

export const type = (obj) => {}
export const isArrayLike = (obj) => {}
export const each = (obj, callback) => {};

В конечном модуле я их подключаю так
import {type, isArrayLike, each} from './Helpers';
Далее в коде я их использую, все три. Но если с isArrayLike и type все в порядке - они сами по себе используются в each, то на первую выходит ошибка:

modules\Helpers.js
67:5  error  'each' is assigned a value but never used  no-unused-vars

Но я ее 100% использую.
Боролся до этого с этим явлением так:

/*eslint-disable no-unused-vars*/
export const extend = function (first) {}
/*eslint-enable no-unused-vars*/

Но это очень неудобно. Подскажите пожалуйста, что я делаю не так?

Answer 1

[Валерий Чупурнов]

Собственно решение, если кто столкнулся:

rules: [
            {
                test: /\.js$/,
                loader: 'babel-loader',
                exclude: /(node_modules|bower_components)/,
            },
            {
                test: /\.js$/,
                loader: 'eslint-loader',
                include: [
                    path.resolve(__dirname, "src"),
                ],
            },
        ]

Меняем местами babel и eslint загрузчики в webpack.config.js

Answer 2

[Николай]

Eslint смотрит пофайлово, в этом модуле вызова нет, вот он и ругается. В офф. Доке(eslint.org/docs/rules/no-unused-vars) написано что с этим делать. Ну и вообще чем вас Array.forEach не устроил?

Comments

[Константин Китманов]

И Array.isArray.

[Валерий Чупурнов]

each может принимать не только массив но и хеш, удобнее чем постоянно с Object.keys.
Функция называется likeArray, я в коде использую фишку
var a = {};
a.push = Array.prototype.push;
и с любым объектом можно работать как с массивом, даже тем который его прямо не наследует. Соответственно проверяем на "массивность" по косвенным признакам.

[Валерий Чупурнов]

Николай Спасибо за наводку, eslint действительно смотрит пофайлово, и его просто надо было подключить в правилах вебпака, после babel

[Николай]

Валерий Чупурнов: вообще это очень плохая идея. Я бы вам предложил просто вот это прочитать: eslint.org/docs/rules/no-unused-vars#exported

[Валерий Чупурнов]

Николай: Почему плохая? Линт показывает и строку и файл по карте.
Я разумеется это читал, и до вашего комментария. Не понимаю чем
/* exported variableName */
намного удобнее чем
/*eslint-disable no-unused-vars*/
/*eslint-enable no-unused-vars*/
лишний код, синтаксис которого запоминать нет ни малейшего желания.

[Николай]

Очевидно, что во втором случае вы намеренно отключаете какие-то правила. В первом же случае вы просто подсказываете линтеру о том, что переменная экспортируется, что вполне обычная практика.

[Валерий Чупурнов]

Николай: Я понимаю что это официальный хак. Ок.
Но вы не ответили почему мое решение "очень плохая идея", все работает, никаких дополнительных правил добавлять не нужно. Проверка на unused работает, eslint проверяет весь файл который компанует babel, а потом по карте показывает куда смотреть. Пока ошибок не заметил, unused показывает очень точно.

[Николай]

Валерий Чупурнов: дело в том, что если вы например захотите сделать вашу часть кода каким-то отдельным бандлом - jslint сломается. У вас появились неявные зависимости. Это не "официальный хак", хак - это что-то плохое, а здесь - нормальный workflow. Как, когда был es5 необходимо было сообщать о глобальных переменных, если они были.

[Валерий Чупурнов]

Николай: Понятно. Отдельно файл использовать не буду, поэтому мне удобнее так, но буду иметь ввиду. Благодарю за ответ.

[Николай]

Валерий Чупурнов: ещё кстати вспомнил, во время написания кода jslint будет ругаться, если он конечно подключен.

[Валерий Чупурнов]

Николай: Я раньше использовал jslint, а что разве eslint это не более новая альтернатива? Зачем использовать их обоих?

[Николай]

Валерий Чупурнов: описка, eslint конечно.