Less.js: [Задача обслуживания] - проверьте настройки ESLint и устраните проблемы с линтингом.

Созданный на 21 июн. 2018  ·  6Комментарии  ·  Источник: less/less.js

Некоторое время назад я переключил проект с JSHint на ESLint (последний лучше поддерживается и предлагает больше функций). Однако я установил многие настройки как предупреждения, потому что у меня не было времени на устранение потенциальных проблем. У меня не было времени и на настройку линтинга, чтобы сделать код более удобным в сопровождении.

Итак, задача, по сути, состояла бы в том, чтобы посмотреть на предупреждения ESLint и посмотреть, что можно безопасно решить. (Например, многие предупреждения о равенстве == не могут быть "исправлены" до === без проверки каждого значения, которое проходит через эту проверку.) Было бы полезно также установить предупреждения для вещи, которые нужно решить в будущем / в настоящее время, например, обеспечение соблюдения документации JSDoc.

good first issue medium priority up-for-grabs

Все 6 Комментарий

Привет, я хотел бы взять это!

@umuur Давай!

@ matthew-dean Хотим ли мы сохранить текущую конфигурацию eslint?
Я не уверен, что нужен TypeScript в качестве парсера и плагина.

@umuur

Хотим ли мы сохранить текущую конфигурацию eslint?

По-разному. Если вы хотите внести изменения, не влияющие на форматирование кода, ничего страшного. Но что касается предупреждений, ИМО, все это действительные предупреждения, и мы либо хотим их устранить, либо оставить комментарий кода об отключенном eslint и (что наиболее важно ) о том, почему он там недействителен.

Хотим ли мы сохранить текущую конфигурацию eslint?

Несмотря на то, что кодовая база еще не использует TypeScript, по моему опыту, он лучше справляется с переносом, чем Babel. Что касается ESLint ... 🤔 да, вы правы, технически он не нужен, пока не будет кода на TypeScript. Вероятно, я сделал это в ожидании преобразования кодовой базы в TS, но теперь я не знаю, произойдет ли / когда это произойдет.

@umuur

Кстати, везде, где вы можете добавить комментарии JSDoc с соответствующими типами параметров, добавьте их!

@ matthew-dean спасибо за подробный комментарий! Сообщу вам об обновлениях.

Была ли эта страница полезной?
0 / 5 - 0 рейтинги