ru-center-group / eslint-config-rucenter

Конфиг eslint для фронтэнд-разработки в Ru-Center Group.
1 stars 2 forks source link

некоторые замечания #3

Open Fernir opened 7 years ago

Fernir commented 7 years ago

Я бы убрал:

  1. Dangerous property 'dangerouslySetInnerHTML' found
  2. Assignment to property of function parameter (иногда хотелось бы менять передаваемый объект, понимаю, что не хороший тон, но кода реально меньше и проще читается)
  3. Unexpected string concatenation (` реально неудобно лично мне. стринги " для атрибутов и ' c \' для js)
  4. Arrow function should not return assignment (иногда ловит довольно удачные комбинации)
pyatyispyatil commented 7 years ago
  1. Да, надо убрать. Уже грядут кейсы, где это по идее придется использовать.
  2. Можно сделать no-param-reassign: ["error", { "props": false }]. Всё же мутабельность у нас в ходу.
  3. Так там вроде отсекается только конкатенация шаблонных строк (http://eslint.org/docs/rules/no-useless-concat). Или это другое правило?
  4. Так можно экранировать скобками: () => (a = 123)

Займусь, как найду время.

evgenykochetkov commented 7 years ago

Поддерживаю, особенно насчет no-param-reassign: ["error", { "props": false }]

Fernir commented 7 years ago

итого "no-param-reassign": ["error", { "props": false }], "react/no-danger": "off", "max-len": "off"

Fernir commented 7 years ago

еще как-то поменять бы no-useless-escape иначе если есть регулярка и в ней экранируешь слеш - ругань

pyatyispyatil commented 7 years ago

Вроде всё добавил. На экранирование добавил ворнинг.