Do not reference tslib.es6.js from package.json exports #208
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
andrewbranch
commented
Jun 2, 2023
| "module": { | ||
| "types": "./tslib.d.ts", | ||
| "default": "./tslib.es6.js" | ||
| "types": "./tslib/modules/index.d.ts", |
Member
Author
There was a problem hiding this comment.
This is where it would be great if we could say "types": "./tslib.d.ts?type=module"
This is a little bit of a lie but I don’t think it matters, for now 🤔 The most correct thing would be to fully duplicate tslib.d.ts to tslib.d.mts.
RyanCavanaugh
approved these changes
Jun 2, 2023
weswigham
reviewed
Jun 2, 2023
| "types": "./tslib.d.ts", | ||
| "default": "./tslib.es6.js" | ||
| "types": "./modules/index.d.ts", | ||
| "default": "./tslib.es6.mjs" |
Member
There was a problem hiding this comment.
Do we have a test for a runtime whose conditions actually load this?
Member
Author
There was a problem hiding this comment.
vite and webpack-5 in the test directory should; possibly others
renovate bot
referenced
this pull request
in valora-inc/wallet
Jun 7, 2023
[](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [tslib](https://www.typescriptlang.org/) ([source](https://togithub.com/Microsoft/tslib)) | [`^2.4.1` -> `^2.5.3`](https://renovatebot.com/diffs/npm/tslib/2.5.0/2.5.3) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>Microsoft/tslib</summary> ### [`v2.5.3`](https://togithub.com/microsoft/tslib/releases/tag/v2.5.3): tslib 2.5.3 [Compare Source](https://togithub.com/Microsoft/tslib/compare/2.5.2...v2.5.3) #### What's Changed - Do not reference tslib.es6.js from package.json exports by [@​andrewbranch](https://togithub.com/andrewbranch) in [https://github.com/microsoft/tslib/pull/208](https://togithub.com/microsoft/tslib/pull/208) **Full Changelog**: microsoft/tslib@2.5.2...v2.5.3 ### [`v2.5.2`](https://togithub.com/microsoft/tslib/releases/tag/2.5.2): tslib 2.5.2 [Compare Source](https://togithub.com/Microsoft/tslib/compare/2.5.1...2.5.2) This release [explicitly re-exports helpers](https://togithub.com/microsoft/tslib/pull/204) to work around TypeScript's incomplete symbol resolution for tslib. ### [`v2.5.1`](https://togithub.com/microsoft/tslib/releases/tag/2.5.1): tslib 2.5.1 [Compare Source](https://togithub.com/Microsoft/tslib/compare/2.5.0...2.5.1) This release of tslib provides fixes for two issues. First, [it reverses the order of `init` hooks provided by decorators](https://togithub.com/microsoft/tslib/pull/202) to correctly reflect proposed behavior. Second, it [corrects the `exports` field of tslib's `package.json` and provides accurate declaration files](https://togithub.com/microsoft/tslib/pull/202) so that it may be consumed under the `node16` and `bundler` settings for `moduleResolution.` </details> --- ### Configuration 📅 **Schedule**: Branch creation - "after 5pm,every weekend" in timezone America/Los_Angeles, Automerge - "after 5pm,every weekend" in timezone America/Los_Angeles. 🚦 **Automerge**: Enabled. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://app.renovatebot.com/dashboard#github/valora-inc/wallet). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNS4xMTAuMCIsInVwZGF0ZWRJblZlciI6IjM1LjExMC4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9--> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Fixes regression from #201