You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This script rebuilds the backend, but inconsistently saves it into node_modules if it already exists. You can tell because running npm run build-backend will result in a message along the lines of:
up to date, audited 782 packages in 899ms (if lc3-backend is not resaved)
changed 1 package, and audited 782 packages in 2s (if lc3-backend is resaved)
This is annoying for development, but doesn't affect the build process, because the GitHub Actions starts off with a fresh node_modules.
Some solutions can be done to patch this for development:
Not using --install-links in the build script
Instead using pnpm i lc3-backend@../backend
Manually removing the package from node_modules before dev
(1) breaks builds because it symlinks lc3-backend into node_modules (which causes Electron Forge + Vite to freak)
(2) requires pnpm (and may also break builds(?))
(3) is annoying
The text was updated successfully, but these errors were encountered:
This script rebuilds the backend, but inconsistently saves it into
node_modules
if it already exists. You can tell because runningnpm run build-backend
will result in a message along the lines of:up to date, audited 782 packages in 899ms
(iflc3-backend
is not resaved)changed 1 package, and audited 782 packages in 2s
(iflc3-backend
is resaved)This is annoying for development, but doesn't affect the build process, because the GitHub Actions starts off with a fresh
node_modules
.Some solutions can be done to patch this for development:
--install-links
in the build scriptpnpm i lc3-backend@../backend
node_modules
before dev(1) breaks builds because it symlinks
lc3-backend
intonode_modules
(which causes Electron Forge + Vite to freak)(2) requires
pnpm
(and may also break builds(?))(3) is annoying
The text was updated successfully, but these errors were encountered: