steroids/dev#804 upgrade streoids/eslint-config #412
Conversation
|
Как связан этот мр с задачей, на которую он ссылается? |
Задачи, на обновление конфигурации eslint'a до последней версии нет. Мне оказалось сподручно обновить и его, пока обновлял react. Это MR и MR на обновление реакта независимы друг от друга. Т.к. Это не совсем по задаче, сделал это в отдельном МР. Если для этого нужна отдельная задача, могу создать. |
|
Пара моментов:
|
Это изменения, приводящие код в соответствие с правилами линтера. Pipeline с ошибкой завершится по lint стадии |
|
Теперь понятно. Тогда создай, пожалуйста, в гитлабе под это задачу, и реф задачи укажи в этих двух MRах |
ProFIT1337
changed the title
| import _isFunction from 'lodash-es/isFunction'; | ||
| import _isEqual from 'lodash-es/isEqual'; | ||
| import _uniqBy from 'lodash-es/uniqBy'; | ||
|
|
There was a problem hiding this comment.
Нужно вернуть возможность отделять импорты пустой строкой, в идеале привести к такому виду
- импорты из реакта, стероидов и сторонних библиотек
// перенос строки
- импорты из корневых папок проекта, для них должен быть алиас, чтобы можно было не указывать весь путь
// перенос строки
- импорты внутри папки
Мне кажется, так сразу будет понятно, откуда что импортируется.
import React from 'react';
import useBem from '@steroidsjs/core/hooks/useBem';
import {DateFormatter} from '@steroidsjs/core/ui/format';
import {Text} from '@steroidsjs/core/ui/typography';
import {Link} from '@steroidsjs/core/ui/nav';
import BaseGrid from 'shared/BaseGrid';
import Button from './views/Button;Подобное мы делали в Клиницисте, можно посмотреть тут
…le to start of ResourceComponent.ts
|
|
||
| export const reInit = () => init(); | ||
|
|
||
| // eslint-disable-next-line default-param-last |
There was a problem hiding this comment.
Почему дизейблится правило, а не меняется местами параметр?
|
|
||
| export const showNotification = ( | ||
| message: string, | ||
| // eslint-disable-next-line default-param-last |
| } | ||
| } | ||
|
|
||
| } |
| @@ -1,4 +1,5 @@ | |||
| /* eslint-disable @typescript-eslint/ban-ts-comment */ | |||
| /* eslint-disable no-promise-executor-return */ | |||
There was a problem hiding this comment.
Ошибки в каком коде отключает это правило, и почему оно включено для всего файла?
| lang: this.language || locale.language, | ||
| }, | ||
| // @ts-ignore | ||
| // eslint-disable-next-line |
There was a problem hiding this comment.
Какую ошибку выключает эта и следующая команда?
| // import {CKEditor} from '@ckeditor/ckeditor5-react'; | ||
| // import ClassicEditor from '@steroidsjs/ckeditor5/packages/ckeditor5-build-classic'; |
| bem.block( | ||
| <div className={bem(bem.block( | ||
| { | ||
| loading: props.isLoading || props.list?.isLoading, | ||
| size: props.size, | ||
| alternatingColors: props.hasAlternatingColors, | ||
| }, | ||
| ), props.className, | ||
| )} | ||
| ), props.className)} |
3 participants
No description provided.