-
Notifications
You must be signed in to change notification settings - Fork 102
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
feat(devtool): Ability to pass custom devtool option to config #2063
Conversation
f0d27ca
to
1380eec
Compare
@@ -117,7 +119,7 @@ export const createConfig = ({ | |||
const devServerPort = typeof port === 'number' ? port : useProxy || standalone ? 1337 : 8002; | |||
return { | |||
mode: mode || (isProd ? 'production' : 'development'), | |||
devtool: false, | |||
devtool: devtool || 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.
The default value is set in the argument now you can simply use devtool: devtool,
or just devtool,
.
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.
I got a bit too caught up in the meeting, thank you!
1380eec
to
478fe59
Compare
:soon::shipit::octocat: |
🌱 🌸 🌷 🌻 🌟 New version of package has been released 🌟 🌻 🌷 🌸 🌱 The release is available on: :package:@redhat-cloud-services/frontend-components-config/v/6.2.9📦 :boom:This feature is brought to you by probot🚀 |
Because apps now use fec as the default config, I want to add the ability to pass in a devtool option. With this we can pass in 'hidden-source-map' for apps that will use sentry. Eventually I want this to just be the default value but I want to use advisor as a test application first.
Now that more of the sentry efforts are unblocked this can move along much faster.
RedHatInsights/insights-advisor-frontend#1268 related PR