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

chore(deps): update babel to v7.24.8 #16472

Merged
merged 4 commits into from
Jul 12, 2024
Merged

chore(deps): update babel to v7.24.8 #16472

merged 4 commits into from
Jul 12, 2024

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Jul 12, 2024

Mend Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
@babel/generator (source) 7.24.7 -> 7.24.8 age adoption passing confidence
@babel/parser (source) 7.24.7 -> 7.24.8 age adoption passing confidence
@babel/types (source) 7.24.7 -> 7.24.8 age adoption passing confidence

Release Notes

babel/babel (@​babel/generator)

v7.24.8

Compare Source

👓 Spec Compliance
🐛 Bug Fix
💅 Polish
babel/babel (@​babel/parser)

v7.24.8

Compare Source


Configuration

📅 Schedule: Branch creation - "after 01:00 on sunday" (UTC), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about these updates again.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Mend Renovate. View repository job log here.

@renovate renovate bot added the dependency Dependency Upgrade label Jul 12, 2024
Copy link

github-actions bot commented Jul 12, 2024

Size Change: +584 B (+0.01%)

Total Size: 7.71 MB

Filename Size Change
./dist/plugins/acorn.js 150 kB -29 B (-0.02%)
./dist/plugins/acorn.mjs 150 kB -29 B (-0.02%)
./dist/plugins/babel.js 314 kB +256 B (+0.08%)
./dist/plugins/babel.mjs 314 kB +256 B (+0.08%)
./dist/plugins/estree.js 199 kB +152 B (+0.08%)
./dist/plugins/estree.mjs 198 kB +152 B (+0.08%)
./dist/plugins/flow.js 662 kB -29 B (0%)
./dist/plugins/flow.mjs 662 kB -29 B (0%)
./dist/plugins/meriyah.js 121 kB -29 B (-0.02%)
./dist/plugins/meriyah.mjs 120 kB -29 B (-0.02%)
./dist/plugins/typescript.js 883 kB -29 B (0%)
./dist/plugins/typescript.mjs 883 kB -29 B (0%)
ℹ️ View Unchanged
Filename Size
./dist/bin 4.1 kB
./dist/bin/prettier.cjs 2.1 kB
./dist/doc.d.ts 6.91 kB
./dist/doc.js 53 kB
./dist/doc.mjs 49.3 kB
./dist/index.cjs 35.8 kB
./dist/index.d.ts 26.6 kB
./dist/index.mjs 768 kB
./dist/internal 4.1 kB
./dist/internal/cli.mjs 125 kB
./dist/LICENSE 206 kB
./dist/package.json 6.28 kB
./dist/plugins 4.1 kB
./dist/plugins/acorn.d.ts 109 B
./dist/plugins/angular.d.ts 177 B
./dist/plugins/angular.js 44.4 kB
./dist/plugins/angular.mjs 43.7 kB
./dist/plugins/babel.d.ts 419 B
./dist/plugins/estree.d.ts 11 B
./dist/plugins/flow.d.ts 90 B
./dist/plugins/glimmer.d.ts 93 B
./dist/plugins/glimmer.js 143 kB
./dist/plugins/glimmer.mjs 142 kB
./dist/plugins/graphql.d.ts 93 B
./dist/plugins/graphql.js 43.6 kB
./dist/plugins/graphql.mjs 43 kB
./dist/plugins/html.d.ts 139 B
./dist/plugins/html.js 151 kB
./dist/plugins/html.mjs 151 kB
./dist/plugins/markdown.d.ts 127 B
./dist/plugins/markdown.js 148 kB
./dist/plugins/markdown.mjs 148 kB
./dist/plugins/meriyah.d.ts 93 B
./dist/plugins/postcss.d.ts 121 B
./dist/plugins/postcss.js 151 kB
./dist/plugins/postcss.mjs 151 kB
./dist/plugins/typescript.d.ts 96 B
./dist/plugins/yaml.d.ts 90 B
./dist/plugins/yaml.js 122 kB
./dist/plugins/yaml.mjs 122 kB
./dist/README.md 4.03 kB
./dist/standalone.d.ts 1.37 kB
./dist/standalone.js 79.3 kB
./dist/standalone.mjs 79.1 kB

compressed-size-action

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Size Change: +584 B (+0.01%)

Total Size: 7.71 MB

Filename Size Change
./dist/plugins/acorn.js 150 kB -29 B (-0.02%)
./dist/plugins/acorn.mjs 150 kB -29 B (-0.02%)
./dist/plugins/babel.js 314 kB +256 B (+0.08%)
./dist/plugins/babel.mjs 314 kB +256 B (+0.08%)
./dist/plugins/estree.js 199 kB +152 B (+0.08%)
./dist/plugins/estree.mjs 198 kB +152 B (+0.08%)
./dist/plugins/flow.js 662 kB -29 B (0%)
./dist/plugins/flow.mjs 662 kB -29 B (0%)
./dist/plugins/meriyah.js 121 kB -29 B (-0.02%)
./dist/plugins/meriyah.mjs 120 kB -29 B (-0.02%)
./dist/plugins/typescript.js 883 kB -29 B (0%)
./dist/plugins/typescript.mjs 883 kB -29 B (0%)
ℹ️ View Unchanged
Filename Size
./dist/bin 4.1 kB
./dist/bin/prettier.cjs 2.1 kB
./dist/doc.d.ts 6.91 kB
./dist/doc.js 53 kB
./dist/doc.mjs 49.3 kB
./dist/index.cjs 35.8 kB
./dist/index.d.ts 26.6 kB
./dist/index.mjs 768 kB
./dist/internal 4.1 kB
./dist/internal/cli.mjs 125 kB
./dist/LICENSE 206 kB
./dist/package.json 6.28 kB
./dist/plugins 4.1 kB
./dist/plugins/acorn.d.ts 109 B
./dist/plugins/angular.d.ts 177 B
./dist/plugins/angular.js 44.4 kB
./dist/plugins/angular.mjs 43.7 kB
./dist/plugins/babel.d.ts 419 B
./dist/plugins/estree.d.ts 11 B
./dist/plugins/flow.d.ts 90 B
./dist/plugins/glimmer.d.ts 93 B
./dist/plugins/glimmer.js 143 kB
./dist/plugins/glimmer.mjs 142 kB
./dist/plugins/graphql.d.ts 93 B
./dist/plugins/graphql.js 43.6 kB
./dist/plugins/graphql.mjs 43 kB
./dist/plugins/html.d.ts 139 B
./dist/plugins/html.js 151 kB
./dist/plugins/html.mjs 151 kB
./dist/plugins/markdown.d.ts 127 B
./dist/plugins/markdown.js 148 kB
./dist/plugins/markdown.mjs 148 kB
./dist/plugins/meriyah.d.ts 93 B
./dist/plugins/postcss.d.ts 121 B
./dist/plugins/postcss.js 151 kB
./dist/plugins/postcss.mjs 151 kB
./dist/plugins/typescript.d.ts 96 B
./dist/plugins/yaml.d.ts 90 B
./dist/plugins/yaml.js 122 kB
./dist/plugins/yaml.mjs 122 kB
./dist/README.md 4.03 kB
./dist/standalone.d.ts 1.37 kB
./dist/standalone.js 79.3 kB
./dist/standalone.mjs 79.1 kB

compressed-size-action

@fisker fisker merged commit bfcfdcc into main Jul 12, 2024
29 checks passed
@fisker fisker deleted the renovate/babel branch July 12, 2024 08:15
) {
return true;
}
break;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sosukesuzuki I just realized this change may affect MDN docs, I don't think people use this feature, but MDN does, should we revert?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Josh-Cena I decided to add () to all nodes in for...in initializer, since there are broken cases, I thought no body actually use this feature, but MDN docs does, is it fine for you to ignore?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, not a problem for us. We can either ignore or just live with it. Thanks for notifying me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependency Dependency Upgrade
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants