Skip to content

Update babel-plugin-proposal-unicode-property-regex for Unicode v12#9636

Merged
nicolo-ribaudo merged 2 commits intomasterfrom
unicode-12
Mar 5, 2019
Merged

Update babel-plugin-proposal-unicode-property-regex for Unicode v12#9636
nicolo-ribaudo merged 2 commits intomasterfrom
unicode-12

Conversation

@mathiasbynens
Copy link
Contributor

@mathiasbynens mathiasbynens commented Mar 5, 2019

@nicolo-ribaudo nicolo-ribaudo added the PR: Spec Compliance 👓 A type of pull request used for our changelog categories label Mar 5, 2019
@babel-bot
Copy link
Collaborator

babel-bot commented Mar 5, 2019 •
edited
Loading

Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/10386/

@buildsize
Copy link

buildsize bot commented Mar 5, 2019 •
edited
Loading

File name Previous Size New Size Change
babel-preset-env.js 2.04 MB 2.04 MB 0 bytes (0%)
babel-preset-env.min.js 1.09 MB 1.09 MB 0 bytes (0%)
babel.js 2.74 MB 3.63 MB 913.56 KB (33%)
babel.min.js 1.53 MB 2.2 MB 689.96 KB (44%)

@mathiasbynens
Copy link
Contributor Author

mathiasbynens commented Mar 5, 2019

Where does the babel.js increase of 912.83 KB come from? Any ideas?

The new Unicode data takes less than a KB (and that's uncompressed). Compare https://bundlephobia.com/result?p=regenerate-unicode-properties@7.0.0 vs. https://bundlephobia.com/result?p=regenerate-unicode-properties@8.0.0.

@nicolo-ribaudo
Copy link
Member

nicolo-ribaudo commented Mar 5, 2019

I'm investigating the size regression

@nicolo-ribaudo
Copy link
Member

nicolo-ribaudo commented Mar 5, 2019 •
edited
Loading

Before:
schermata da 2019-03-05 17-24-37

After:
schermata da 2019-03-05 17-28-36

EDIT: For some reason the top-level version of regexpu-core is 4.4.0, so lerna can't hoist regexpu-core@4.5.2. I have manually updated the lock file.

@nicolo-ribaudo
Copy link
Member

nicolo-ribaudo commented Mar 5, 2019

Fixed
schermata da 2019-03-05 18-00-00

@nicolo-ribaudo nicolo-ribaudo merged commit 349c0d4 into master Mar 5, 2019
@nicolo-ribaudo nicolo-ribaudo deleted the unicode-12 branch March 5, 2019 17:06
@danez
Copy link
Member

danez commented Mar 5, 2019 •
edited
Loading

Should we also update unicode in the parser and regenerate the identifier regexes?

Okay I missed the other PR.

@danez
Copy link
Member

danez commented Mar 5, 2019 •
edited
Loading

This makes babel-standalone fail on node 6

WARNING in ./node_modules/regenerate-unicode-properties/Binary_Property/Grapheme_Base.js
Module build failed: RangeError: Maximum call stack size exceeded
    at Object.needsParens (/home/travis/build/babel/babel/node_modules/@babel/generator/lib/node/index.js:112:10)
    at Generator.print (/home/travis/build/babel/babel/node_modules/@babel/generator/lib/printer.js:287:25)
    at Generator.MemberExpression (/home/travis/build/babel/babel/node_modules/@babel/generator/lib/generators/expressions.js:261:8)
    at withSource (/home/travis/build/babel/babel/node_modules/@babel/generator/lib/printer.js:299:22)
    at Buffer.withSource (/home/travis/build/babel/babel/node_modules/@babel/generator/lib/buffer.js:185:28)
    at Generator.withSource (/home/travis/build/babel/babel/node_modules/@babel/generator/lib/printer.js:178:15)
    at Generator.print (/home/travis/build/babel/babel/node_modules/@babel/generator/lib/printer.js:298:10)
    at Generator.CallExpression (/home/travis/build/babel/babel/node_modules/@babel/generator/lib/generators/expressions.js:177:8)
    at withSource (/home/travis/build/babel/babel/node_modules/@babel/generator/lib/printer.js:299:22)
    at Buffer.withSource (/home/travis/build/babel/babel/node_modules/@babel/generator/lib/buffer.js:185:28)
 @ ./node_modules/regenerate-unicode-properties ^\.\/.*\.js$
 @ ./node_modules/regexpu-core/rewrite-pattern.js
 @ ./packages/babel-plugin-proposal-unicode-property-regex/lib/index.js
 @ ./packages/babel-standalone/src/index.js

It does not fail travis unfortunately, which we should fix.

@nicolo-ribaudo
Copy link
Member

nicolo-ribaudo commented Mar 5, 2019 •
edited
Loading

This looks like a bug with Babel itself? 🤔
Maybe it is #8278 (comment)

@mathiasbynens
Copy link
Contributor Author

mathiasbynens commented Mar 6, 2019

Was about to reference #8278 here myself.

mathiasbynens added a commit to mathiasbynens/regenerate-unicode-properties that referenced this pull request Mar 6, 2019
@mathiasbynens
Avoid deeply-nested ASTs
f76769f 
babel/babel#8278
babel/babel#9636
mathiasbynens added a commit to mathiasbynens/regexpu-core that referenced this pull request Mar 6, 2019
@mathiasbynens
Update dependencies
aac2e6d 
babel/babel#9636
@mathiasbynens
Copy link
Contributor Author

mathiasbynens commented Mar 6, 2019

I just published regenerate-unicode-properties@8.0.1 which changes the generated output to avoid deeply-nested ASTs. regexpu-core@4.5.3 has the updated dependency.

mathiasbynens added a commit that referenced this pull request Mar 6, 2019
@mathiasbynens
Update regexpu-core dependency
a96101e 
yarn upgrade regexpu-core@4.5.3

This should resolve the issue @danez mentions here:
#9636 (comment)
@mathiasbynens mathiasbynens mentioned this pull request Mar 6, 2019
Merged
mathiasbynens added a commit that referenced this pull request Mar 6, 2019
@mathiasbynens
Update regexpu-core dependency
22bf9de 
yarn upgrade regexpu-core@4.5.3

This should resolve the issue @danez mentions here:
#9636 (comment)
mathiasbynens added a commit that referenced this pull request Mar 7, 2019
@mathiasbynens
Update regexpu-core dependency
f501146 
yarn upgrade regexpu-core@4.5.3

This should resolve the issue @danez mentions here:
#9636 (comment)
@danez danez mentioned this pull request Mar 7, 2019
Merged
mathiasbynens added a commit that referenced this pull request Mar 7, 2019
@mathiasbynens
Update regexpu-core dependency (#9642)
7c909e8 
This should resolve the issue @danez mentions here:
#9636 (comment)
@lock lock bot added the outdated A closed issue/PR that is archived due to age. Recommended to make a new issue label Oct 4, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Oct 4, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@xtuc xtuc xtuc approved these changes

Assignees

No one assigned

Labels

outdated A closed issue/PR that is archived due to age. Recommended to make a new issue PR: Dependency ⬆️ PR: Spec Compliance 👓 A type of pull request used for our changelog categories

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@mathiasbynens @babel-bot @nicolo-ribaudo @danez @xtuc