Skip to content

Conversation

@beeme1mr
Copy link
Member

This PR

  • Includes types in the package.json export

Related Issues

Fixes #354

Notes

Thanks for reporting the bug @talzion12

Signed-off-by: Michael Beemer beeme1mr@users.noreply.github.com

Signed-off-by: Michael Beemer <beeme1mr@users.noreply.github.com>
@beeme1mr beeme1mr requested a review from toddbaert as a code owner December 23, 2022 18:08
@beeme1mr beeme1mr linked an issue Dec 23, 2022 that may be closed by this pull request
@codecov-commenter
Copy link

codecov-commenter commented Dec 23, 2022

Codecov Report

Merging #355 (5c5dc78) into main (9fbb2bd) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #355   +/-   ##
=======================================
  Coverage   99.67%   99.67%           
=======================================
  Files          15       15           
  Lines        1236     1236           
  Branches       97       97           
=======================================
  Hits         1232     1232           
  Misses          4        4           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Signed-off-by: Michael Beemer <beeme1mr@users.noreply.github.com>
@beeme1mr
Copy link
Member Author

Documentation around dual CommonJS/ES modules:
https://nodejs.org/api/packages.html#dual-commonjses-module-packages

Copy link
Member

@toddbaert toddbaert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Documentation around dual CommonJS/ES modules: https://nodejs.org/api/packages.html#dual-commonjses-module-packages

This is pretty disappointing, but at least it's documented.

@toddbaert toddbaert merged commit 11f8efe into main Dec 23, 2022
lukas-reining pushed a commit that referenced this pull request Jun 29, 2023
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@beeme1mr beeme1mr deleted the 354-bug-typescript-doesnt-find-the-included-types branch February 5, 2025 21:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Typescript doesn't find the included types

4 participants