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

Migration from 17.3.0 to 18.0.0 - Cannot find module '@scaleleap/amazon-marketplaces' or its corresponding type declarations #634

Closed
xhuberdeau opened this issue May 13, 2024 · 3 comments · Fixed by #635
Assignees
Labels

Comments

@xhuberdeau
Copy link

xhuberdeau commented May 13, 2024

Hello,

Thanks for this repository!

I'm trying to migrate this dependency from 17.3.0 to 18.0.0 but typescript shows me an error:
image.

I digged a bit hoping this might help: the difference I see in the node_modules between 17.3.0 and 18.0.0 is that now, the lib files are not directly in the lib folder but in a src subfolder. So the types property in the package.json seems wrong as it's pointing to a non-existing file (it should be lib/src/index.d.ts instead of lib/index.d.ts)

Version 17.3.0

image

Version 18.0.0

image

Tell me if you need more information 🙂

nguyentoanit added a commit that referenced this issue May 13, 2024
There was a bug when deleting `@scaleleap/utils` package from our
repository.
Including `"test/**/*.ts"` into `tsconfig.json` leaded to loosing entry
points in the build.


https://github.com/ScaleLeap/amazon-marketplaces/pull/619/files#diff-b55cdbef4907b7045f32cc5360d48d262cca5f94062e353089f189f4460039e0R12


Closes #634 ;
github-actions bot pushed a commit that referenced this issue May 13, 2024
## [18.0.1](v18.0.0...v18.0.1) (2024-05-13)

### Bug Fixes

* entry points in the build ([#635](#635)) ([9ceab9a](9ceab9a)), closes [/github.com//pull/619/files#diff-b55cdbef4907b7045f32cc5360d48d262cca5f94062e353089f189f4460039e0R12](https://github.com//github.com/ScaleLeap/amazon-marketplaces/pull/619/files/issues/diff-b55cdbef4907b7045f32cc5360d48d262cca5f94062e353089f189f4460039e0R12) [#634](#634)
Copy link

🎉 This issue has been resolved in version 18.0.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@nguyentoanit
Copy link
Contributor

@xhuberdeau: This issue should be addressed in v18.0.1. 🚀

@xhuberdeau
Copy link
Author

@xhuberdeau: This issue should be addressed in v18.0.1. 🚀

Wow, thanks for the quick fix 🎉

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 a pull request may close this issue.

2 participants