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: react bump #144

Merged
merged 11 commits into from
Sep 17, 2024
Merged

feat: react bump #144

merged 11 commits into from
Sep 17, 2024

Conversation

effektivnayarabota1
Copy link
Contributor

closes #140

@effektivnayarabota1
Copy link
Contributor Author

@Nelfimov
в npm не нашел версию 18.2.2, поставил ^18.2.0 - в devDeps & peerDeps

Copy link
Member

@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.

упали чеки

packages/upload/package.json Show resolved Hide resolved
packages/upload/package.json Outdated Show resolved Hide resolved
@effektivnayarabota1
Copy link
Contributor Author

@Nelfimov
тайпчек висит, на локалке тоже.
у нас же с shdvor была похожая ошибка?
image

@Nelfimov
Copy link
Member

сноси yarn.lock, переустанавливай зависимости, убирай дубликаты зависимостей

@effektivnayarabota1
Copy link
Contributor Author

@Nelfimov много ошибок линтера и тайпчека. поместить эту в другую задачу?

  • была ошибка линтера - убрал eslint из корневого package.json
  • ошибку typecheck - убрал, бампнув @types/node до 22й версии
  • бампнул @types/react - до последней
  • добавил @types/tldjs в app-links - потомучто была ошибка сборки пакета над которым работаю

  • сейчас много release-ошибок - это я чтото наколдовал, или после твоих обновлений добавились проверки типов на релизе?

@Nelfimov
Copy link
Member

@Nelfimov много ошибок линтера и тайпчека. поместить эту в другую задачу?

Линт - в другую отдельную. Не приоритет.

Тайпчек - в рамках этой

  • сейчас много release-ошибок - это я чтото наколдовал, или после твоих обновлений добавились проверки типов на релизе?

Это новый бандл и окружение. Тайпчек и релизы надо фиксить.

@effektivnayarabota1
Copy link
Contributor Author

@Nelfimov
работали над пакетом @atls/react-upload

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

@Nelfimov Nelfimov merged commit 9770fa1 into master Sep 17, 2024
3 of 6 checks passed
@Nelfimov Nelfimov deleted the feat/bump-react branch September 17, 2024 07:06
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.

Грейд версии @atls/react-upload
2 participants