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

chore(root): update package.json. Update workspaces list #138

Merged
merged 3 commits into from Mar 16, 2023

Conversation

nickhsine
Copy link
Contributor

@nickhsine nickhsine commented Mar 15, 2023

Remove packages/embed-code-generator pkg from workspaces. The reason we remove it is because we want packages/embed-code-generator to have its own node_modules. We don't want it to use the symbolic link to other subpackages.

Since we might update other subpackages at the same time, if the codes in other subpackages are not committed and not packaged. embed-code-generator might mistakenly import these ongoing pkgs while building the webpack bundles.

@nickhsine
Copy link
Contributor Author

Remove `packages/embed-code-generator` pkg from `workspaces`.
The reason we remove it is because we want `packages/embed-code-generator`
to have its own `node_modules`. We don't want it to use the
symbolic link to other subpackages.

Since we might update other subpackages at the same time,
if the codes in other subpackages are not committed and not packaged.
embed-code-generator might mistakenly import these undergoing pkgs
while building the webpack bundles.
@nickhsine
Copy link
Contributor Author

因為專案上線時程壓力,我先 merge 這個 PR,之後會再補上 README。

@nickhsine nickhsine merged commit bec1729 into readr-media:main Mar 16, 2023
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

1 participant