[main][doc] add document for how to fix type error in extensions #2395
Conversation
|
|
||
| ## Reason | ||
|
|
||
| When a new version of applicationinsights-web (which includes applicationinsights-core as a dependency) is released, package management tools may automatically update applicationinsights-core to the new version. However, if the extensions (e.g., React, Angular) have not been updated, their dependencies on applicationinsights-core may not match, leading to type errors. |
There was a problem hiding this comment.
It would be good to call out here that this is a TypeScript issue only as we work really hard to ensure that while the "types" (might) appear different to different versions of TypeScript at runtime the code should still interoperate as long as the "core" version is later than any sub-component as we support backward compatibility but not forward compatibility.
Which means we support older components working with "newer" dependencies (ie the newer dependency is backward compatible for older components) but not newer components working with "older" dependencies.
A good example of this is that we may from time to time "add" new public helper functions to core components that some "newer" extensions might use / need. As such that "newer" function is only available in the "newer" version of the core component and not in the older previous published components. (ie. we don't know now (or back then) what functions we "might" need in a future release -- why I call it forward compatibility)
| ``` | ||
| ### 3. Examples to follow | ||
| Here is a discussion that provide deeper insights into resolving these issues: | ||
| https://github.com/microsoft/applicationinsights-react-js/issues/95 |
There was a problem hiding this comment.
I think there is a new TypeScript flag (which most people wont use) -
- skipLibCheck
- noCheck
I thought I read recently (somewhere) that there was a new config but I can't find it currently
There was a problem hiding this comment.
Thanks. Will check that potential walk around before merge this pr.
microsoft/applicationinsights-react-js#95