Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat/update cra #6

Merged
merged 3 commits into from
Nov 9, 2021
Merged

Feat/update cra #6

merged 3 commits into from
Nov 9, 2021

Conversation

Powenga
Copy link

@Powenga Powenga commented Nov 9, 2021

Описание

Обновил версию cra установив react-scripts последней версии
Установил eslint 7.32.0 (последняя версия 8, но react-scripts не работает с ней)
Установил все что нужно для airbnb
"eslint-config-airbnb": "^18.2.1",
"eslint-plugin-import": "^2.25.2",
"eslint-plugin-jsx-a11y": "^6.4.1",
"eslint-plugin-react": "^7.26.1",
"eslint-plugin-react-hooks": "^4.2.0";
Сам eslint-config-airbnb
Добавил переменные окружения, чтобы не падало приложение от ошибок eslint

Проверил npm install, все работает, но, думаю, еще раз скачать ветку и проверить лишним не будет.

Ссылка на задачу

Сделать pr с настройками линтера

@boredcity
Copy link

👍 все работает

прямо тут в PR оставлю эту ссылку на issue, просто чтоб вы и коллеги могли подписаться и следить за развитием событий -- тогда можно будет поднять версию, если разработчики CRA поддержат версию 🕵️

можно перед влитием прогнать npx eslint . --fix, который все что может сам починит (в основном, trailing запятые и скобочки) 👍

Copy link

@boredcity boredcity left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

если ни у кого не появится в самое ближайшее время возражений, можно вливать и обвинять во всем меня 👍

кстати, полторы полезные идеи:

  1. рассказывать в PR как его тестировать (запустили приложение, проверили что во всех файлах ругается на вещи, не соответствующие набору правил, прогнали npx eslint .)
  2. проставлять теги (Labels), которые заранее скажут, что изменились зависимости и нужно сделать npm ci в ветке

@Powenga Powenga added the dependencies Dependencies updated (npm ci) label Nov 9, 2021
Copy link

@gremwiz1 gremwiz1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Скачал к себе локально ветку - все работает.

@gremwiz1 gremwiz1 merged commit abfeb90 into main Nov 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Dependencies updated (npm ci)
Projects
None yet
3 participants