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

Cleanup #452

Closed
wants to merge 4 commits into from
Closed

Cleanup #452

wants to merge 4 commits into from

Conversation

ITStan
Copy link

@ITStan ITStan commented Aug 12, 2020

No description provided.

@ITStan
Copy link
Author

ITStan commented Aug 12, 2020

@unpete обнаружена непонятная странность
rollup как target имеет cjs
в то время как сверху запускается webpack с target umd
По идее umd поддерживается всеми, и CJS и AMD загрузчиками.
Но вот только нативные импорты не умеют работать в UMD.
То есть по хорошему нужно перебрать все чтобы получить совместимость с esm.

@unpete
Copy link
Member

unpete commented Aug 16, 2020

Корень репозитоия не используется. Исходники живут в /packages и публикуются средствами lerna.
Соглашусь, что рефакторинг и реструктуризация нужны. Они будут глубокими, изменим состав и функциональную нагрузку библиотек. Задача пока в очереди.

@unpete unpete closed this Aug 16, 2020
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.

None yet

2 participants