Skip to content

chore: add missing @babel/runtime dependencies - #32954

Merged
LekoArts merged 2 commits into
gatsbyjs:masterfrom
ascorbic:chore/babel-runtime
Aug 30, 2021
Merged

chore: add missing @babel/runtime dependencies#32954
LekoArts merged 2 commits into
gatsbyjs:masterfrom
ascorbic:chore/babel-runtime

Conversation

@ascorbic

@ascorbic ascorbic commented Aug 29, 2021

Copy link
Copy Markdown
Contributor

This PR adds @babel/runtime as a dependency to all packages that are missing it. Because packages are all compiled with @babel/plugin-transform-runtime, they all need it as a dependency. It usually works ok because it is a dependency of gatsby, but it's good practice to not rely on transitive dependencies, and in some situations can be tree-shaken away.

@gatsbot gatsbot Bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Aug 29, 2021
@ascorbic
ascorbic force-pushed the chore/babel-runtime branch from 03d23ac to 245bb2a Compare August 29, 2021 18:11
@ascorbic ascorbic changed the title chore: add missing dependencies chore: add missing @babel/runtime dependencies Aug 29, 2021
@wardpeet wardpeet added type: maintenance An issue or pull request describing a change that isn't a bug, feature or documentation change and removed status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer labels Aug 30, 2021
@ascorbic

Copy link
Copy Markdown
Contributor Author

The test failures seem to be unrelated (and seem to be a weird snapshot failing which actually is identical)

@LekoArts
LekoArts merged commit 401b358 into gatsbyjs:master Aug 30, 2021
@ascorbic
ascorbic deleted the chore/babel-runtime branch August 30, 2021 13:05
pragmaticpat pushed a commit to pragmaticpat/gatsby that referenced this pull request Apr 28, 2022
Co-authored-by: gatsbybot <mathews.kyle+gatsbybot@gmail.com>
raffishquartan pushed a commit to raffishquartan/gatsby that referenced this pull request Apr 28, 2026
Co-authored-by: gatsbybot <mathews.kyle+gatsbybot@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: maintenance An issue or pull request describing a change that isn't a bug, feature or documentation change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants