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

Cannot find module './getBigInt' #837

Closed
mrobit opened this issue May 17, 2022 · 8 comments · Fixed by #838
Closed

Cannot find module './getBigInt' #837

mrobit opened this issue May 17, 2022 · 8 comments · Fixed by #838
Labels

Comments

@mrobit
Copy link

mrobit commented May 17, 2022

Apologies if this has been answered before or I'm just not understanding it, but memfs@^3.4.1 as a peer dependency of webpack-dev-middleware has an issue where the lib/Stats.js fails to import ./getBigInt – looking through node_modules/memfs the getBigInt.js file is located at lib/src/getBigInt.js instead of lib/getBigInt.js

Again, sorry if this has been addressed before or I'm not doing it right. Just wanted to see if this has been patched somewhere and I'm a little confused. For now I've reverted to 3.4.1 in my project's package-lock.json so we're alright for now, but wanted to post this issue in here in case it was at all helpful. Thanks!

@Abu-Zakaria
Copy link

I got the same issue a couple of minutes ago.

@patrickconroy
Copy link

I just ran into this as well, tracked it down to this version.

@mrobit
Copy link
Author

mrobit commented May 17, 2022

Didn't realize that 3.4.2 just got release 33 min ago 😂 glad to see it's not just me.

@G-Rath
Copy link
Collaborator

G-Rath commented May 17, 2022

I'll try and look into this now but can't give an ETA sorry - it might have to wait until at least the weekend

@conlanpatrek
Copy link

conlanpatrek commented May 17, 2022

If it's helpful, the last version's npm package is borked. Looks like somehow getBigInt.js is winding up in lib/src/getBigInt.js but the import path ends up looking in lib/getBigInt.js. It's only happening with that one file, the rest of the files end up in lib just fine.

EDIT: I see where mrobit already included these details. Guess that's what I get for skimming the e-vite. My mistake.

@G-Rath
Copy link
Collaborator

G-Rath commented May 17, 2022

It's because of #818, as cpy is what is used to copy getBigInt.js into lib - currently trying to figure out what actually has changed to cause this as nothing from the release notes (that I've tried) jump out as being the actual cause.

@streamich
Copy link
Owner

🎉 This issue has been resolved in version 3.4.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

@kelyar
Copy link

kelyar commented May 17, 2022

thanks for quick fix

@streamich streamich unpinned this issue Jun 22, 2023
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.

7 participants