-
Notifications
You must be signed in to change notification settings - Fork 28
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
Update DCR to TS 5.1 along will relevant packages #8094
Conversation
Size Change: -219 kB (-26%) 🎉 Total Size: 625 kB
ℹ️ View Unchanged
|
…pdate-guardian-packages
…pdate-guardian-packages
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.
+1 LGTM
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.
Thanks for doing this Joe! 🌟
Just a couple of questions.
"tslib": "2.5.0", | ||
"typescript-json-schema": "0.54.0", | ||
"tslib": "2.5.3", | ||
"type-fest": "3.12.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.
Would you be able to point me to where this is used? I can't find any references to it 🤔
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.
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.
@guardian/commercial lists it as a dev dep but not a peer dep - @arelra might be something to look at?
What does this change?
Upgrades DCR to TS 5.1 all bumps all corresponding deps in line with peer dependencies.
Due to a number of issues with some Guardian packages being hoisted to the root dir while others stay in the project folders I've disabled hoisting for all @guardian packages.
In turn this has highlighted a number of issues with missing project dependencies which have needed resolving.
In short, the key changes are as follows:
node-fetch
andmockdate
dep to ar