-
-
Notifications
You must be signed in to change notification settings - Fork 300
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
deps: update typescript and lint dependencies #5995
Conversation
Performance Report✔️ no performance regression detected Full benchmark results
|
"command": { | ||
"version": { | ||
"message": "chore(release): %s" | ||
} | ||
} | ||
}, | ||
"$schema": "node_modules/lerna/schemas/lerna-schema.json" |
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.
why remove the schema?
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.
It's added, not removed. There is a command lerna repair
which formats and fix the lerna config file automatically.
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.
May want to add a note the PR body that some exports were changed to type exports
That's has no effect on the usage of those types. A lot of these were done in other PR where we added |
"useWorkspaces": true, | ||
"useNx": true, |
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.
Were those removed because by default they are se to true
anyways?
🎉 This PR is included in v1.12.0 🎉 |
Motivation
Keep the dependencies up-to-date.
Description
Steps to test or reproduce