steroids/dev#804 updated steroids/eslint-config #253
Conversation
|
Обновление версии реакта привело к изменениям в 61 файле? o_O |
Так же как в react-core. Я создал два MR. Один только обновляет |
ProFIT1337
changed the title
| {...props} | ||
| {...props.pickerProps} | ||
| className={bem(bem.block({ranged: isRange}), props.className)} | ||
| // eslint-disable-next-line react/no-unstable-nested-components |
There was a problem hiding this comment.
Какую ошибку прячет этот коммент?
| const MenuItemView = props.itemView; | ||
|
|
||
| const renderMenuItems = React.useCallback(() => ( | ||
| // eslint-disable-next-line react/jsx-no-useless-fragment |
There was a problem hiding this comment.
Это точно бест практис? Может как-то можно вернуть фрагмент без отрубания правил линта?
| ref={props.forwardedRef} | ||
| className={bem( | ||
| bem.block({ | ||
| className={bem(bem.block({ |
There was a problem hiding this comment.
Будто бы стало менее читаемо. Какое lint-правило заставляет так писать?
| return props.renderList( | ||
| <div className={bem( | ||
| bem.block( | ||
| <div className={bem(bem.block( |
There was a problem hiding this comment.
Такой же случай как и выше, стало менее читаемо
| bem: any, | ||
| onClick: () => void, | ||
| iconName: string, | ||
| // eslint-disable-next-line default-param-last |
There was a problem hiding this comment.
Зачем дизейблить правило? Мне кажется оно тут правильно подсказывает.
| key={props.index} | ||
| className={bem( | ||
| bem.block( | ||
| className={bem(bem.block( |
2 participants
No description provided.