react-redux-starter-kit icon indicating copy to clipboard operation
react-redux-starter-kit copied to clipboard

Прикрутить prettier

Open Znack opened this issue 5 years ago • 10 comments

Сразу настроить нормально приттиер, чтобы ручками не форматировать и чтобы кодстайл консистентный был

Znack avatar Apr 02 '19 04:04 Znack

Предлагаю в контексте этой же задачи перейти с tslint на eslint и подружить eslint с prettier.

Тут есть неплохая инструкция как это сделать https://levelup.gitconnected.com/setting-up-eslint-with-prettier-typescript-and-visual-studio-code-d113bbec9857. Плюс там же есть настройки для VSCode, для автофикса/автоформата на сохранение файла. Локально попробовал, понравилось :)

Еще нужно будет для prettier конфиг написать, чтобы длину строки поправить, юзать одинарные кавычки, возможно что-то еще. Ну и для eslint возможно тоже нужно будет правила некоторые поправить.

in19farkt avatar Oct 04 '19 09:10 in19farkt

кстати да, перейти на eslint стоит, но я бы сделал отдельной таской :)

Znack avatar Oct 04 '19 10:10 Znack

А то дифф и так огромный, лучше чтобы можно было четко выделить диффы чисто на форматтинг, либо чисто на правки по линтингу

Znack avatar Oct 04 '19 10:10 Znack

кстати да, перейти на eslint стоит, но я бы сделал отдельной таской :)

почему еще не завели ишью для этого?

prodderman avatar Oct 15 '19 09:10 prodderman

Почему нет? вот же она https://github.com/fullstack-development/react-redux-starter-kit/issues/130 :)

in19farkt avatar Oct 15 '19 09:10 in19farkt

я думаю нужно сначала сделать и мержнуть #130, после нее прикручивать prettier и дружить его с eslint

in19farkt avatar Oct 15 '19 09:10 in19farkt

Можно эту таску начинать, за подробностями можно мне в личку, уже есть рабочий конфиг и интеграция с eslint

in19farkt avatar Nov 12 '19 10:11 in19farkt

Закрыл ПР, давайте заново просто тогда

Znack avatar Nov 12 '19 10:11 Znack

tslint is depreceted, typescript has moved to eslint

safr avatar Jan 17 '20 13:01 safr

@Safr дык мы уже как бэ :)

in19farkt avatar Jan 17 '20 13:01 in19farkt