-
-
Notifications
You must be signed in to change notification settings - Fork 2k
[CHORE]: Remove Babel from stackgl_modules build process #7862
Copy link
Copy link
Closed
Labels
P2considered for next cycleconsidered for next cyclechoreTrack maintenance work (dependency bumps, build/CI, internal refactors, docs, etc.)Track maintenance work (dependency bumps, build/CI, internal refactors, docs, etc.)good for agentplotly-internalsize: 1taskone-off taskone-off task
Description
Activity
Metadata
Metadata
Assignees
Labels
P2considered for next cycleconsidered for next cyclechoreTrack maintenance work (dependency bumps, build/CI, internal refactors, docs, etc.)Track maintenance work (dependency bumps, build/CI, internal refactors, docs, etc.)good for agentplotly-internalsize: 1taskone-off taskone-off task
Description
Remove Babel-related dependencies from the
stackgl_modulesbuild process.Notes
stackgl_modulesis a pre-bundled vendor folderstackgl_modulesbuild process has used Babel to transpile the included dependencies to ES5 syntaxScope
In
stackgl_modules/webpack.config.js:babel-loaderrules (the project-files rule is effectively a no-op without a babel config; thebuffer/is-mobilerule is redundant with esbuild)target: ['web', 'es5']In
stackgl_modules/package.json, drop the now-unused devDeps:@babel/core@babel/preset-env@babel/plugin-transform-modules-commonjsbabel-loaderify-loader(glslify) andraw-loadermust stay — they handle the GLSL shader preprocessing that's the actual reason this pre-bundle exists.Testing
npm run bundle-stackgl