Skip to content
This repository was archived by the owner on Feb 13, 2023. It is now read-only.

Conversation

@sethidden
Copy link
Contributor

@sethidden sethidden commented Jun 2, 2022

this commit was made because after moving the whole repo from packages/theme to
the template-magento repo unit tests stopped working

this is because all packages in magento2 repo are in the repo root's
node_modules, so packages/theme doesn't contain them normally. this changes in
the context of template-magento

also reverted some changes made during the rc7 release which attempted to fix
the issue

this pr aligns template-magento, whereas #24 aligns magento2 repo

this commit was made because after moving the whole repo from packages/theme to
the template-magento repo unit tests stopped working

this is because all packages in magento2 repo are in the repo root's
node_modules, so packages/theme doesn't contain them normally. this changes in
the context of template-magento

also reverted some changes made during the rc7 release which attempted to fix
the issue
@Frodigo Frodigo merged commit 661c368 into main Jun 2, 2022
@Frodigo Frodigo deleted the M2-639 branch June 2, 2022 14:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants