Skip to content
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

Update babelrc and add sourcemap to production #409

Merged
merged 1 commit into from
May 24, 2017
Merged

Update babelrc and add sourcemap to production #409

merged 1 commit into from
May 24, 2017

Conversation

gauravtiwari
Copy link
Member

@gauravtiwari gauravtiwari commented May 24, 2017

  • [Babel] Dynamic import() and Class Fields and Static Properties babel plugin to .babelrc
  • Enable source-map in production

@gauravtiwari gauravtiwari changed the title Minor babelrc and sourcemap update Update babelrc and add sourcemap to production May 24, 2017
@dhh dhh merged commit 3dd186a into rails:master May 24, 2017
@gauravtiwari gauravtiwari deleted the minor-updates branch May 24, 2017 14:13
@swrobel
Copy link
Contributor

swrobel commented Jan 24, 2018

I was just trying to figure out why sourcemaps are being generated in production and came across this, so at least I know it's intentional. What's the rationale behind enabling them in prod?

Edit: I see it's using nosources-source-map so the actual pre-Uglify source isn't included, just filenames & line numbers (I think). Here's the PR that made that change: #770

No need to respond at this point!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants