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

webpack upgrade #255

Closed
Nelfimov opened this issue Sep 15, 2023 · 11 comments · Fixed by #277
Closed

webpack upgrade #255

Nelfimov opened this issue Sep 15, 2023 · 11 comments · Fixed by #277
Assignees
Labels
feature New feature or request Stale

Comments

@Nelfimov
Copy link
Member

С чем связан запрос на фичу?

Необходимо прогрейдить версию webpack до 5 версии

Расскажите как вы это себе видите

webpack-chain поддерживает только создание конфигурации для webpack версии 4. Поддержка 5-ой версии есть только в форках - neutrinojs/webpack-chain#297.

Необходимо либо найти альтернативу либо использовать форком (нежелательно).

Приложите пример реализаций

No response

@Nelfimov Nelfimov added the feature New feature or request label Sep 15, 2023
@TorinAsakura TorinAsakura changed the title webpack: переход на версию 5 webpack upgrade Sep 16, 2023
@Nelfimov
Copy link
Member Author

  • Конфигурацию webpack обновил для 5 версии
  • Убрал использование webpack-chain - с 2020 нет обновлений и поддержки 5 версии
  • Протестировал на своем репозитории: https://github.com/Nelfimov/odin-battleship/tree/test/yarn-webpack

Тестирование проводил так:

  • в atls/tools:
    • билд @atls/code-service
    • билд @atls/code-service-worker
    • билд нового yarn бандла
  • в тестируемом репозитории подключил наш yarn
  • подключил @atls/code-service
  • запустил yarn service build
  • проверил наличие всех импортированных ассетов в билде (шрифты, картинки, иконки)
  • сравнил с old_dist - билд с использованием старого вебпака из пет проекта

Что заметил

Билд что с текущим yarn и @atls/code-service что с рефактором производит один и тот же бандл - шрифты включаются.

@TorinAsakura стоит ли тогда "трогать, если работает"?

Ошибка с шрифтами в наших проектах связана с тем, что injectFontFaces была неправильно написана. Для этого в hyperion обновил инструкцию и протестировал. Конфигурация нашего вебпака на вывод не влияет - next использует свой вебпак + swc.

@TorinAsakura
Copy link
Member

@Nelfimov Переход на 5-й вебпак так или иначе неизбежен, ведь мы стремимся к микрофронтам, а без 5-о вебпака их не реализовать через федерации

@TorinAsakura
Copy link
Member

@Nelfimov На данной стадии, если это видится геморроем - можешь не трогать и оставить задачку для бэклога

@Nelfimov
Copy link
Member Author

@Nelfimov На данной стадии, если это видится геморроем - можешь не трогать и оставить задачку для бэклога

Конфиг переписан, сборка у обычных реакт приложений проходит успешно. Тестировал на репе выше.

Может еще где-то протестировать? Например, https://github.com/atls/reactjs/tree/master/packages/payment-widget

@archeon-atls
Copy link
Contributor

Статус?

@Nelfimov
Copy link
Member Author

Nelfimov commented Oct 3, 2023

Конфиг переписан нативными средствами.

Договорились изучить вопрос микро-фронтендов, через ModuleFederation

@archeon-atls archeon-atls removed the Stale label Oct 4, 2023
@TorinAsakura
Copy link
Member

Так чо с таской-то? Закрываем? Или чего?

@Nelfimov
Copy link
Member Author

Nelfimov commented Oct 5, 2023

Я планирую сделать тестовую репу с микро фронтами для проверки конфига.

@TorinAsakura
Copy link
Member

Я планирую сделать тестовую репу с микро фронтами для проверки конфига.

У нас есть прожект-стартер - вполне можно там обкатать, двойную пользу сделаем

@archeon-atls
Copy link
Contributor

Статус?

@archeon-atls
Copy link
Contributor

Таске месяц. Закрыта за утратой актуальности.

@archeon-atls archeon-atls closed this as not planned Won't fix, can't repro, duplicate, stale Nov 20, 2023
@TorinAsakura TorinAsakura reopened this Nov 20, 2023
@Nelfimov Nelfimov linked a pull request Dec 22, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request Stale
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants