Skip to content

Comments

fix: import path of networkConfig#7694

Merged
twoeths merged 1 commit intopeerDASfrom
te/fix_network_config_import
Apr 13, 2025
Merged

fix: import path of networkConfig#7694
twoeths merged 1 commit intopeerDASfrom
te/fix_network_config_import

Conversation

@twoeths
Copy link
Contributor

@twoeths twoeths commented Apr 13, 2025

Motivation

Error [ERR_UNSUPPORTED_DIR_IMPORT]: Directory import '/usr/src/lodestar/packages/beacon-node/lib/network/subnets' is not supported resolving ES modules imported from /usr/src/lodestar/packages/beacon-node/lib/network/networkConfig.js
Apr 13 08:18:20 devnet-ax41-0 beacon_run.sh[2274894]:     at finalizeResolution (node:internal/modules/esm/resolve:254:11)
Apr 13 08:18:20 devnet-ax41-0 beacon_run.sh[2274894]:     at moduleResolve (node:internal/modules/esm/resolve:921:10)
Apr 13 08:18:20 devnet-ax41-0 beacon_run.sh[2274894]:     at defaultResolve (node:internal/modules/esm/resolve:1120:11)
Apr 13 08:18:20 devnet-ax41-0 beacon_run.sh[2274894]:     at ModuleLoader.defaultResolve (node:internal/modules/esm/loader:557:12)
Apr 13 08:18:20 devnet-ax41-0 beacon_run.sh[2274894]:     at ModuleLoader.resolve (node:internal/modules/esm/loader:526:25)
Apr 13 08:18:20 devnet-ax41-0 beacon_run.sh[2274894]:     at ModuleLoader.getModuleJob (node:internal/modules/esm/loader:249:38)
Apr 13 08:18:20 devnet-ax41-0 beacon_run.sh[2274894]:     at ModuleJob._link (node:internal/modules/esm/module_job:126:49) {
Apr 13 08:18:20 devnet-ax41-0 beacon_run.sh[2274894]:   code: 'ERR_UNSUPPORTED_DIR_IMPORT',
Apr 13 08:18:20 devnet-ax41-0 beacon_run.sh[2274894]:   url: 'file:///usr/src/lodestar/packages/beacon-node/lib/network/subnets'

Description

  • should not import folder path, import specific files instead

@twoeths twoeths requested a review from a team as a code owner April 13, 2025 08:28
@twoeths twoeths merged commit 8f80d36 into peerDAS Apr 13, 2025
13 of 17 checks passed
@twoeths twoeths deleted the te/fix_network_config_import branch April 13, 2025 08:41
@wemeetagain
Copy link
Member

🎉 This PR is included in v1.34.0 🎉

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.

3 participants