-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Remove peer dependency on @babel/core
from most packages
#2985
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
--- | ||
'@emotion/babel-plugin': patch | ||
'@emotion/css': patch | ||
'@emotion/native': patch | ||
'@emotion/primitives': patch | ||
'@emotion/react': patch | ||
'@emotion/styled': patch | ||
--- | ||
|
||
Remove peer dependency on `@babel/core` |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -18,7 +18,6 @@ | |
], | ||
"dependencies": { | ||
"@babel/helper-module-imports": "^7.16.7", | ||
"@babel/plugin-syntax-jsx": "^7.17.12", | ||
"@babel/runtime": "^7.18.3", | ||
"@emotion/hash": "^0.9.0", | ||
"@emotion/memoize": "^0.8.0", | ||
|
@@ -30,9 +29,6 @@ | |
"source-map": "^0.5.7", | ||
"stylis": "4.1.3" | ||
}, | ||
"peerDependencies": { | ||
"@babel/core": "^7.0.0" | ||
}, | ||
Comment on lines
-33
to
-35
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This feels off but I guess that it's not a big deal if we omit it from here - we mostly use Babel through what we are given by the caller anyway. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. i'm also still curious though - is this removal technically needed to fix the mentioned issues? Isn't inling There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This comment indirectly answers this question. This is somewhat needed today because we have a direct dependency on |
||
"devDependencies": { | ||
"@babel/core": "^7.18.5", | ||
"babel-check-duplicated-nodes": "^1.0.0" | ||
|
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.
this depends on
@babel/types
but I guess that it should be mostly OK since those are just factories for simple objects and they shouldn't rely on identity of any class or anything like that