-
-
Notifications
You must be signed in to change notification settings - Fork 400
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
feat(minifier): fold empty array constructors #6236
base: 10-01-feat_minifier_fold_object_constructor
Are you sure you want to change the base?
feat(minifier): fold empty array constructors #6236
Conversation
Your org has enabled the Graphite merge queue for merging into mainAdd the label “0-merge” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix. You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link. |
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @camchenry and the rest of your teammates on Graphite |
CodSpeed Performance ReportMerging #6236 will not alter performanceComparing Summary
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good. Any suggestions for approches supporting the normalize
in closure compiler as confirguration? It seems necessary in this ast pass.
I'm not sure really, I'd imagine that this should be something that is done to modify the AST prior to starting the minification. |
34b67dc
to
5298d6e
Compare
a32b3f1
to
b9dfb56
Compare
5298d6e
to
42704fe
Compare
No description provided.