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

fix(common): yarn check #184

Merged
merged 21 commits into from
Jan 15, 2025
Merged

fix(common): yarn check #184

merged 21 commits into from
Jan 15, 2025

Conversation

kirill-ivanovvv
Copy link
Contributor

@kirill-ivanovvv kirill-ivanovvv commented Jan 10, 2025

to #102

@kirill-ivanovvv kirill-ivanovvv self-assigned this Jan 10, 2025
@kirill-ivanovvv kirill-ivanovvv marked this pull request as ready for review January 15, 2025 11:00
@kirill-ivanovvv
Copy link
Contributor Author

kirill-ivanovvv commented Jan 15, 2025

@Nelfimov

  • большую часть ошибок закрыл
  • сильно глубокие типы не трогал, перекрыл исключениями
  • старые исключения не трогал
  • как промежуточный вариант, работы там еще много

вот этот момент не понял как поправить . ошибка парсинга next.config.js - на /* eslint-disable */ не реагирует

image

@Nelfimov
Copy link
Contributor

вот этот момент не понял как поправить . ошибка парсинга next.config.js - на /* eslint-disable */ не реагирует

Надо эти файлы в tsconfig добавить. Либо include либо exclude

Copy link
Contributor

@Nelfimov Nelfimov left a comment

Choose a reason for hiding this comment

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

релизы упали

@kirill-ivanovvv
Copy link
Contributor Author

вот этот момент не понял как поправить . ошибка парсинга next.config.js - на /* eslint-disable */ не реагирует

Надо эти файлы в tsconfig добавить. Либо include либо exclude

оба варианта попробовал, не работает

@Nelfimov
Copy link
Contributor

оба варианта попробовал, не работает

Попробуй их в ts перевести - вроде nextjs позволяет

Copy link
Contributor

@Nelfimov Nelfimov left a comment

Choose a reason for hiding this comment

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

релиз упал. Попробуй прогрейдить некст до 15

@kirill-ivanovvv
Copy link
Contributor Author

релиз упал. Попробуй прогрейдить некст до 15

это может быть изза того, что реакт 18? его тоже поднимать?

image

@Nelfimov
Copy link
Contributor

Нет, думаю просто не доглядели key

@kirill-ivanovvv
Copy link
Contributor Author

@Nelfimov

  • предлагаю мерджить с этими двумя ошибками
  • бамп некста вынесу в отдельную таску
  • эти две ошибки вынесу в отдельную таску, свяжу с бампом некста
  • как пофиксить верстку - сейчас не могу понять, key не помог.

image

@Nelfimov Nelfimov merged commit b2849af into master Jan 15, 2025
5 of 6 checks passed
@Nelfimov Nelfimov deleted the fix/yarn-check branch January 15, 2025 17:08
@Nelfimov
Copy link
Contributor

  • эти две ошибки вынесу в отдельную таску, свяжу с бампом некста

Эту таску на потом тогда - сейчас остатки по шдвору и к другим проектам

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants