search

fullstack-development / react-redux-starter-kit

Modular starter kit for React+Redux+React Router projects.
https://demo.fullstack-development.com/
MIT License
91 stars 13 forks source link

130. Migrating to eslint #151

Closed prodderman closed 4 years ago

prodderman commented 4 years ago

Перевел правила tslint в eslint по возможности. Заюзал Димину основу для конфига. Появилось куча ошибок по типу: модуль такой-то должен быть в dependencies, а не в devDependencies, или метод такой-то дожен быть объявлен перед методом таким-то (потому что используются стрелочные методы в некоторых местах), или использование глобальных require и прочие ошибки форматирования. Мне их в этой задаче править?

in19farkt commented 4 years ago

@prodderman да, в этом ПР исправляй все ошибки линтинга.

in19farkt commented 4 years ago

По поводу разделения депсов и девДепсов. Ящитаю нужно делать так:

Такое разбиение мне кажется позволит держать package.json в чистоте, видеть что у нас в рантайме вообще используется.

in19farkt commented 4 years ago

но я не уверен, что сейчас стоит реструктурировать депсы, т.к. у нас еще в параллельных ветках идут работы с зависимостями, будут жесткие конфликты в package-lock. Предлагаю сейчас заигнорить это правило в конфиге, но оставить TODO, чтобы не забыли потом навести порядок там.

in19farkt commented 4 years ago

Еще проверь что нет правила на обязательность префикса I для типов объявленных с помощью interface. Тут https://github.com/fullstack-development/react-redux-starter-kit/issues/150 вроде пришли к тому что это не нужно.

Znack commented 4 years ago

Я думаю, что после каждлого мержа в мастер код должен оставаться в мастере чистым — eslint не ругаться, тс компилиться и тд. Поэтому тут либо вырубить правила пока, которые мы не поддерживаем (возможно с TODO и с обяхательным созданием ишью на восстановления этих правил, причем этим ишью надо будет повышенный приорттет), либо прямо тут с исправлениями, если это не очень сложно

in19farkt commented 4 years ago

я думаю тема с депсами не критичная, поэтому можно заигнорить пока с тудухой. Как более менее зависимости устаканятся, тогда и вернем это правило

in19farkt commented 4 years ago

@prodderman не было проблем с правилом import/no-unresolved ?

Я сейчас столкнулся с ней, еслинт не подхватывает почему-то paths из tsconfig. Решил установкой eslint-import-resolver-typescript и правками в eslintrc:

  settings: {
    'import/resolver': {
      typescript: {
        alwaysTryTypes: true,
      },
    },
  },
in19farkt commented 4 years ago

@prodderman Пачка конфликтов подъехала :)

in19farkt commented 4 years ago

@prodderman какой статус по ПРу? когда сможешь доделать?

kinda-neat commented 4 years ago

image ну, названия уже лучше, это респект. а так, следим за тобой

kinda-neat commented 4 years ago

@prodderman какой статус по ПРу? когда сможешь доделать?

@prodderman вопросы в воздух что ли летят? ты можешь нас ориентировать как-то? чтобы если что мы эту задачу кому-то другому отдали, вот Лехе Ч например, вообще нифига непонятно что у тебя в голове, так что описывай словами, надоело тебе из раза в раз напоминать

prodderman commented 4 years ago

@prodderman какой статус по ПРу? когда сможешь доделать?

@prodderman вопросы в воздух что ли летят? ты можешь нас ориентировать как-то? чтобы если что мы эту задачу кому-то другому отдали, вот Лехе Ч например, вообще нифига непонятно что у тебя в голове, так что описывай словами, надоело тебе из раза в раз напоминать

Передал задачу Лехе @chmnkh

kinda-neat commented 4 years ago

@prodderman какой статус по ПРу? когда сможешь доделать?

@prodderman вопросы в воздух что ли летят? ты можешь нас ориентировать как-то? чтобы если что мы эту задачу кому-то другому отдали, вот Лехе Ч например, вообще нифига непонятно что у тебя в голове, так что описывай словами, надоело тебе из раза в раз напоминать

Передал задачу Лехе @chmnkh

спасибо за ответ спустя неделю и два меншена -_-

chmnkh commented 4 years ago

@kinda-neat @in19farkt готово

@prodderman капец ты блин, неделю переменную переименовывал))

in19farkt commented 4 years ago

@sk1e твой аппрув еще нужен, без него ничего не получится :(

in19farkt commented 4 years ago

еее!!!