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

Прикрутить prettier #104

Open Znack opened 5 years ago

Znack commented 5 years ago

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

in19farkt commented 4 years ago

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

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

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

Znack commented 4 years ago

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

Znack commented 4 years ago

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

prodderman commented 4 years ago

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

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

in19farkt commented 4 years ago

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

in19farkt commented 4 years ago

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

in19farkt commented 4 years ago

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

Znack commented 4 years ago

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

Safr commented 4 years ago

tslint is depreceted, typescript has moved to eslint

in19farkt commented 4 years ago

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