electron-store icon indicating copy to clipboard operation
electron-store copied to clipboard

Upgrade to latest conf

Open ArTiSTiX opened this issue 1 year ago • 2 comments

Hello !

I am trying to use a fork of conf package (in order to test my changes before making a pull request), but electron-store still uses an outdated version of conf. Since conf is now using ESM, it requires some changes which makes the upgrade not-so-trivial.

I tried to do it, but i am loosing my mind, i don't know what i'm doing wrong in either my electron app, or in my changes, but when importing my own upgraded fork of electron-store, im having an error:

Error [ERR_REQUIRE_ESM]: require() of ES Module L:\dev\ljas\midi-jar\node_modules\electron-store\index.js

Here is my fork: https://github.com/la-jarre-a-son/electron-store/tree/upgrade-conf-esm

Could someone be of any help, i've seen some forks using latest conf, but no PR, and tests are not passing.

ArTiSTiX avatar Sep 12 '23 12:09 ArTiSTiX

I guess this is the same issue as for dot-prop, you wait for Electron to be ESM-friendly...

ArTiSTiX avatar Sep 12 '23 14:09 ArTiSTiX

I believe Electron have completed the PR on ESM. is it possible to now update electron-store?

iamterryclark avatar Oct 04 '23 09:10 iamterryclark

did you find any workaround for this? I'm getting the same error and been looking for ways to fix it but no luck

imkimchi avatar Mar 06 '24 10:03 imkimchi

When I packaged and ran my program, I encountered the "conf module not found" issue. I think we might be facing the same problem, perhaps due to conf using ESM. I hope the official team can resolve this.

Attiybom avatar Apr 14 '24 03:04 Attiybom

@sindresorhus the same issue. need a pr?

npmstudy avatar Apr 29 '24 03:04 npmstudy