[BUG] Unmet peer dependency of tslib warning from yarn

[chadsowald]

Description/Screenshot

Similar to #1114 which although is marked as fixed, I still see tslib in peer deps in the package.json here: https://github.com/microsoft/ApplicationInsights-JS/blob/master/shared/AppInsightsCore/package.json#L61

Shouldn't that be moved to devDependencies or just removed all together?

Repro steps:

• Create a new folder
• Run yarn init and accept defaults
• Run yarn add @microsoft/applicationinsights-web
• Notice warnings!

Expected behavior No warnings

[MSNev]

I believe this is an issue with your project in that you will need to include tslib into your package.json.

Internally, we dynamically replace all internal tslib references so we are not dependent or tied to specific versions, which is also why we have it listed as a generic "*" (all version) peerDependency.

We could try removing all peerDependency references, but I believe last time we did this some version (don't remember which) used by some customers complained that we didn't include it and rather than a warning it was an error.

[github-actions[bot]]

This Issue will be closed in 30 days. Please remove the "Stale" label or comment to avoid closure with no action.

[MSNev]

This is still a schedule task, it's just lower priority to try and remove it as a peer dependency so it's kept getting moved.

[github-actions[bot]]

This Issue will be closed in 30 days. Please remove the "Stale" label or comment to avoid closure with no action.