Skip to content
This repository has been archived by the owner on Jan 11, 2023. It is now read-only.

[SourceMaps] update to source-map to 0.7.2 #5598

Closed
andreicristianpetcu opened this issue Mar 4, 2018 · 8 comments
Closed

[SourceMaps] update to source-map to 0.7.2 #5598

andreicristianpetcu opened this issue Mar 4, 2018 · 8 comments

Comments

@andreicristianpetcu
Copy link
Contributor

Hi,

Source-map got new and shiny performance fixes in 0.7.2. It would be really cool if the debugger would benefit from the following fixes:

https://hacks.mozilla.org/2018/01/oxidizing-source-maps-with-rust-and-webassembly/
http://fitzgeraldnick.com/2018/02/26/speed-without-wizardry.html

@claim claim bot added the not-available label Mar 4, 2018
@andreicristianpetcu
Copy link
Contributor Author

Relevant issue in upstream babel/babel#7486

@jasonLaster
Copy link
Contributor

Thanks! Good call

@andreicristianpetcu
Copy link
Contributor Author

andreicristianpetcu commented Mar 4, 2018

Here is a list of dependencies that depend on source-map

babel-generator
babel-cli
babel-core
babel-register

clean-css
csso
devtools-map-bindings
devtools-source-map

handlebars
istanbul-lib-source-maps
jest-cli - needs istanbul-lib-source-maps first
jstransform - cannot open issue, only pull requests
postcss
pretty-fast
recast
snapdragon
source-map-support
uglify-es
uglify-js

uglifyjs-webpack-plugin
webpack-sources
webpack - needs webpack-sources first
inline-source-map
convert-source-map - needs inline-source-map first

Optional dependencies
escodegen

If I missed anything, please feel free to add it.

@andreicristianpetcu
Copy link
Contributor Author

Any idea which one would be more important?

@castdin
Copy link
Contributor

castdin commented Mar 9, 2018

/claim

@claim
Copy link

claim bot commented Mar 9, 2018

Thanks for claiming the issue! 👋

Here are some links for getting setup, contributing, and developing. We're always happy to answer questions in slack! If you become busy, feel free to /unclaim it.

🦊 Debugger team!

@jasonLaster jasonLaster changed the title Please update source-map to 0.7.2 [SourceMaps] update to source-map to 0.7.2 Mar 14, 2018
@castdin
Copy link
Contributor

castdin commented Apr 5, 2018

/unclaim

@claim
Copy link

claim bot commented Apr 5, 2018

Done 👍

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

No branches or pull requests

3 participants