Skip to content

Fix types - #52

Merged
lovell merged 2 commits into
lovell:mainfrom
lcharette:main
Sep 29, 2025
Merged

Fix types#52
lovell merged 2 commits into
lovell:mainfrom
lcharette:main

Conversation

@lcharette

Copy link
Copy Markdown
Contributor

See : #51 (comment)

With 4.2.0 and Typescript enabled on my project, I now get this error because it doesn't find declarations files:

Could not find a declaration file for module 'limax'.  '[...]/node_modules/limax/lib/limax.mjs' implicitly has an 'any' type.
  There are types at '[...]/node_modules/limax/index.d.ts', but this result could not be resolved when respecting package.json "exports". The 'limax' library may need to update its package.json or typings.

4 import slug from 'limax'

@lcharette lcharette mentioned this pull request Sep 29, 2025
@lovell
lovell merged commit 280579c into lovell:main Sep 29, 2025
7 checks passed
@lovell

lovell commented Sep 29, 2025

Copy link
Copy Markdown
Owner

Thank you, I'm going to add a test for this too to help prevent regression.

@lovell

lovell commented Sep 29, 2025

Copy link
Copy Markdown
Owner

Definition tests added via commit 5d451e8

v4.2.1 now available, thanks for reporting and fixing.

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.

2 participants