-
Notifications
You must be signed in to change notification settings - Fork 532
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
Resolve type annotation changes required to enable composite
in tsconfig
#1565
Conversation
97cf4e3
to
d09fbe4
Compare
composite
in tsconfig
d09fbe4
to
b42bb0d
Compare
src/lib/analytics.ts
Outdated
@@ -82,7 +94,8 @@ function postAnalytics(data) { | |||
|
|||
const headers = {}; | |||
if (snyk.api) { | |||
headers.authorization = 'token ' + snyk.api; | |||
// headers.authorization = 'token ' + snyk.api; |
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.
Any reason for this comment to be here?
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.
Nope, I'll remove it. Thanks!
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.
small comment, but LG
b42bb0d
to
714ba35
Compare
Expected release notes (by @maxjeffos) fixes:
|
🎉 This PR is included in version 1.437.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
What does this PR do?
Fixes some typing issues that are needed for the upcoming modular CLI init PR (#1564).
Any background context you want to provide?
For the modular CLI init / monorepo PR, we will be turning on
"composite": true
in the tsconfig which requires certain typing things be addressed beforehand.Why not just do this as part of the aforementioned PR (#1564)? Because that PR is going to touch a lot of things and I would prefer to keep that PR as small as reasonably possible and since this is just about improving some typing (and refactoring analytics into TypeScript) I think it is best kept on its own.
What are the relevant tickets?
HAMMER-237