-
Notifications
You must be signed in to change notification settings - Fork 3
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
Bump Webpack to v5 and node to 18 #343
Changes from all commits
e4fe505
a846bfe
c6d5a94
103a9c2
e2f8033
ee13f21
f9e8bd0
4bd1dc4
5cff68c
cd0232e
56555ed
d3fb56e
6ff926d
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 |
---|---|---|
@@ -1,17 +1,17 @@ | ||
// .babelrc | ||
{ | ||
"presets": [ | ||
["es2015", {"modules": false}], | ||
"react" | ||
], | ||
"plugins": [ | ||
"transform-object-assign", | ||
"transform-class-properties", | ||
"react-hot-loader/babel" | ||
], | ||
"env": { | ||
"test": { | ||
"plugins": ["transform-es2015-modules-commonjs"] | ||
} | ||
} | ||
"@babel/preset-react", | ||
[ | ||
"@babel/preset-env", | ||
{ | ||
"useBuiltIns": "entry", | ||
"corejs": "3.19", | ||
"targets": { | ||
"chrome": 75, | ||
"ios": 12 | ||
} | ||
} | ||
] | ||
] | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1 +1 @@ | ||
12.13.1 | ||
18.18.2 | ||
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. Out of interest, why upgrade to Node 18 over 20? Was this because of issues with 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 file was deleted.
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.
Is this config change needed due to upgrading webpack?
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.
Some of the existing babel dependencies were incompatible - since I was moving from
babel-core
et al to@babel/core
et all I needed to overhaul this file - mostly this is borrowed from Composer's.babelrc