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
Prune and upgrade npm package dependencies #1265
Prune and upgrade npm package dependencies #1265
Conversation
Signed-off-by: Peter Bomber <peter.bomber@microsoft.com>
Signed-off-by: Peter Bomber <peter.bomber@microsoft.com>
Signed-off-by: Peter Bomber <peter.bomber@microsoft.com>
Signed-off-by: Peter Bomber <peter.bomber@microsoft.com>
Signed-off-by: Peter Bomber <peter.bomber@microsoft.com>
Signed-off-by: Peter Bomber <peter.bomber@microsoft.com>
Signed-off-by: Peter Bomber <peter.bomber@microsoft.com>
Signed-off-by: Peter Bomber <peter.bomber@microsoft.com>
Signed-off-by: Peter Bomber <peter.bomber@microsoft.com>
Signed-off-by: Peter Bomber <peter.bomber@microsoft.com>
aeeab9a
to
fd9f566
Compare
@@ -17,7 +17,9 @@ | |||
"noImplicitAny": true, | |||
"strictFunctionTypes": true, | |||
"strictNullChecks": true, | |||
"noImplicitThis": true | |||
"noImplicitThis": true, | |||
"esModuleInterop": 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.
Setting esModuleInterop to true is recommended and required for compatibility with later versions of the K8s client library.
"noImplicitThis": true | ||
"noImplicitThis": true, | ||
"esModuleInterop": true, | ||
"skipLibCheck": 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.
Without setting this, we end up with type errors in the K8s client, the glob types, and shelljs.
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 this. Looks good to me, I gave a quick test for the vsix shared, and seems alright and things work normal. ☕️🙏
For anyone keen here is the vsix
to test if any wider community folks what to chip in. Thanks
Updates dependencies to the latest versions, prior to enabling Dependabot to do this for us.
Change summary:
@types
packages have been removed where packages publish their own typesglob
andrequest
)tsconfig.json
to prevent TypeScript analysing thenode_modules
directory and enable the recommendedesModuleInterop
setting)This builds, runs and passes automated tests, but it's probably worth doing some further manual testing.