-
Notifications
You must be signed in to change notification settings - Fork 296
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
fix(ci): moving several tasks from yarn to bun, improve tests #1566
Conversation
Run & review this pull request in StackBlitz Codeflow. |
✅ Deploy Preview for cornerstone-3d-docs canceled.
|
* @param page - Playwright page object. | ||
*/ | ||
|
||
export const reduceViewportsSize = async (page) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we do 0.4
}node_modules/webpack/bin/webpack.js --progress --config ${webpackConfigPath}` | ||
); | ||
}); | ||
const conf = buildConfig(exampleNames, examplePaths, distDir, validPath(rootPath)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you leave it as is and create another one
@@ -129,7 +128,23 @@ module.exports = { | |||
} | |||
}, | |||
optimization: { | |||
minimize: false | |||
minimize: false, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we need source map for cornerstonejs.org but we can it have for ci
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great stuff
No description provided.