-
Notifications
You must be signed in to change notification settings - Fork 386
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: update more babel deps #3576
Conversation
import MatcherUtils = jest.MatcherUtils; | ||
import CustomMatcherResult = jest.CustomMatcherResult; | ||
const { globSync } = glob; |
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.
Jest errors for some reason now unless we do import * as
@@ -38,7 +38,6 @@ | |||
"@rollup/plugin-replace": "^5.0.2", | |||
"@rollup/plugin-typescript": "^11.1.1", | |||
"@types/babel__core": "^7.20.1", | |||
"@types/glob": "^8.1.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.
No longer needed since glob ships with its own types, and these types were messing us up.
"devDependencies": { | ||
"@babel/core": "^7.22.5", | ||
"@babel/helper-explode-assignable-expression": "^7.18.6", |
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.
Not sure why this happened, but compat mode isn't long for this world, so we'll remove this soon anyway
/nucleus test |
Details
Updates our remaining Babel deps. Follow-up to #3573
Does this pull request introduce a breaking change?
Does this pull request introduce an observable change?