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

Use source-map-js (source-map's fork) #1515

Merged
merged 4 commits into from May 17, 2021
Merged

Use source-map-js (source-map's fork) #1515

merged 4 commits into from May 17, 2021

Conversation

7rulnik
Copy link
Contributor

@7rulnik 7rulnik commented Jan 15, 2021

I made the fork from source-map and named it source-map-js. It contains several performance optimizations. Now source-map consumer should be 4x faster.

PR with optimizations: 7rulnik/source-map-js#2
Reasons and difference of fork: https://github.com/7rulnik/source-map#difference-between-original-source-map

@7rulnik 7rulnik changed the title Use @7rulnik/source-map fork Use source-map-js (source-map's fork) Feb 7, 2021
@7rulnik 7rulnik marked this pull request as ready for review February 7, 2021 22:12
@7rulnik
Copy link
Contributor Author

7rulnik commented Feb 7, 2021

@ai now it's ready :)

papandreou added a commit to assetgraph/assetgraph that referenced this pull request Feb 8, 2021
papandreou added a commit to assetgraph/assetgraph that referenced this pull request Feb 8, 2021
@ai ai changed the base branch from main to murmur May 17, 2021 12:39
@ai ai merged commit 9ac31e8 into postcss:murmur May 17, 2021
@ai
Copy link
Member

ai commented May 21, 2021

Released in 8.3.

Be ready to fix bugs :D

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.

None yet

2 participants