babel / website

:globe_with_meridians: The Babel documentation website
https://babeljs.io/docs/en/index.html
MIT License
746 stars 1.3k forks source link

docs(options.md): clarify `--source-maps` CLI option #2898

Closed jedwards1211 closed 3 days ago

jedwards1211 commented 1 month ago

it's confusing. Honestly it would be less confusing and IMO better if having sourceMaps: true in config files would make @babel/cli write .map files to disk. I don't like the reasoning in https://github.com/babel/babel/issues/2484.

netlify[bot] commented 1 month ago

Deploy request for babel-next pending review.

Visit the deploys page to approve it

Name Link
Latest commit 1ff35fd49e752c94a470eae1a3660ab0c1b896ae
netlify[bot] commented 1 month ago

Deploy Preview for babel ready!

Built without sensitive environment variables

Name Link
Latest commit 1ff35fd49e752c94a470eae1a3660ab0c1b896ae
Latest deploy log https://app.netlify.com/sites/babel/deploys/666398680c6f1f00083c3e65
Deploy Preview https://deploy-preview-2898--babel.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

liuxingbaoyu commented 2 days ago

https://github.com/babel/babel/pull/14950 I remember that we have support for if having sourceMaps: true in config files would make @babel/cli write .map files to disk. I tested it with this and the .map files were generated correctly.

echo '{"sourceMaps": true}' > .babelrc
mkdir src
echo 'export default 1' > src/index.js
babel src --out-dir lib