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

Warning upon building with 4.1.6 (latest) #126

Closed
james-criscuolo opened this issue Feb 20, 2019 · 2 comments
Closed

Warning upon building with 4.1.6 (latest) #126

james-criscuolo opened this issue Feb 20, 2019 · 2 comments

Comments

@james-criscuolo
Copy link

I'm seeing a new warning when building with webpack, it's an angular AOT build:

WARNING in ./node_modules/electron-json-storage/node_modules/write-file-atomic/index.js
Module not found: Error: Can't resolve 'worker_threads' in '/{path_to_my_project}/node_modules/electron-json-storage/node_modules/write-file-atomic'
 @ ./node_modules/electron-json-storage/node_modules/write-file-atomic/index.js
 @ ./node_modules/electron-json-storage/lib/storage.js
 @ {where my import is}

This does not appear to break my build, so it does not look like I'll need to go to an earlier version. I import a non-standard way, but it appears the recommended way also causes it:

// import * as desktopStorage from "electron-json-storage";
const desktopStorage = require("electron-json-storage");

(both cause the issue)
I can also confirm this does not occur with 4.1.5

@jviotti
Copy link
Member

jviotti commented Feb 27, 2019

Hey @james-criscuolo ,

That looks like an issue on write-file-atomic, so I'd suggest re-opening this issue there. I believe it works fine in previous versions because this module started requiring write-file-atomic quite recently!

@james-criscuolo
Copy link
Author

Closing this as Node 12 is now LTS, and it doesn't occur there.

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

No branches or pull requests

2 participants