fix: change packing to use npm pack to included nested node_modules #785

agdimech · 2024-05-23T04:30:45Z

Nested node_modules were being emitted from the tgz that was being published to npm. This should hopefully resolve the issue by performing the following:

Only install prod dependencies when running pnpm deploy
Hoist installed dependencies when running pnpm deploy
Use standard npm pack oppose to pnpm pack.
Added verbose logging for the packing command to verify local vs remote manifest matching.

Fixes #755

fix #755

…odules (#785)" This reverts commit 58497d9.

…odules (#785)" (#786) This reverts commit 58497d9.

fix: change packing to use npm pack to included nested node_modules

7930df9

fix #755

agdimech merged commit 58497d9 into mainline May 23, 2024
4 checks passed

agdimech deleted the fix/755 branch May 23, 2024 22:56

agdimech added a commit that referenced this pull request May 23, 2024

Revert "fix: change packing to use npm pack to included nested node_m…

594c7a8

…odules (#785)" This reverts commit 58497d9.

agdimech added a commit that referenced this pull request May 23, 2024

Revert "fix: change packing to use npm pack to included nested node_m…

6fc5ca1

…odules (#785)" (#786) This reverts commit 58497d9.

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: change packing to use npm pack to included nested node_modules #785

fix: change packing to use npm pack to included nested node_modules #785

agdimech commented May 23, 2024 •

edited

Loading

fix: change packing to use npm pack to included nested node_modules #785

fix: change packing to use npm pack to included nested node_modules #785

Conversation

agdimech commented May 23, 2024 • edited Loading

agdimech commented May 23, 2024 •

edited

Loading