Skip to content
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

fix: remove typesVersions from package.json #219

Merged
merged 1 commit into from
May 27, 2022

Conversation

achingbrain
Copy link
Member

@achingbrain achingbrain commented May 27, 2022

We have typesVersions to allow deep requires from these modules, but
they cause incorrect paths to be detected by tsc (see #214).

Since #161 we control exports using the exports map, and we only export
the root from each module so deep requires are disallowed and consequently
typesVersions isn't necessary.

Since it's causing problems and we don't need it, remove it. We can
revisit once we publish ESM-only but right now this fixes up the types
tsc generates.

Fixes #214

We have `typesVersions` to allow deep imports from these modules, but
they cause incorrect paths to be detected by tsc (see #214).

Since #161 we control exports using the exports map, and we only export
the root from each module so `typesVersions` isn't necessary.

Since it's causing problems and we don't need it, remove it.  We can
revisit once we publish ESM-only but right now this fixes up the types
tsc generates.

Fixes #214
@achingbrain achingbrain merged commit 465670e into master May 27, 2022
@achingbrain achingbrain deleted the fix/remove-types-versions branch May 27, 2022 11:54
github-actions bot pushed a commit that referenced this pull request May 27, 2022
## [ipfs-unixfs-v6.0.9](ipfs-unixfs-v6.0.8...ipfs-unixfs-v6.0.9) (2022-05-27)

### Bug Fixes

* remove typesVersions from package.json ([#219](#219)) ([465670e](465670e)), closes [#214](#214) [#161](#161) [#214](#214)

### Trivial Changes

* update dep-check command ([#221](#221)) ([5802bd3](5802bd3))
@github-actions
Copy link

🎉 This PR is included in version ipfs-unixfs-v6.0.9 🎉

The release is available on:

Your semantic-release bot 📦🚀

github-actions bot pushed a commit that referenced this pull request May 27, 2022
## [ipfs-unixfs-importer-v9.0.10](ipfs-unixfs-importer-v9.0.9...ipfs-unixfs-importer-v9.0.10) (2022-05-27)

### Bug Fixes

* remove typesVersions from package.json ([#219](#219)) ([465670e](465670e)), closes [#214](#214) [#161](#161) [#214](#214)

### Trivial Changes

* update dep-check command ([#221](#221)) ([5802bd3](5802bd3))
@github-actions
Copy link

🎉 This PR is included in version ipfs-unixfs-importer-v9.0.10 🎉

The release is available on:

Your semantic-release bot 📦🚀

github-actions bot pushed a commit that referenced this pull request May 27, 2022
## [ipfs-unixfs-exporter-v7.0.11](ipfs-unixfs-exporter-v7.0.10...ipfs-unixfs-exporter-v7.0.11) (2022-05-27)

### Bug Fixes

* remove typesVersions from package.json ([#219](#219)) ([465670e](465670e)), closes [#214](#214) [#161](#161) [#214](#214)

### Trivial Changes

* update dep-check command ([#221](#221)) ([5802bd3](5802bd3))
@github-actions
Copy link

🎉 This PR is included in version ipfs-unixfs-exporter-v7.0.11 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ipfs-unixfs-importer: import type declaration imports UnixFS from the wrong location
1 participant