Skip to content
This repository has been archived by the owner on Feb 12, 2024. It is now read-only.

jsipfs config show throws error #3192

Closed
jacobheun opened this issue Jul 23, 2020 · 0 comments · Fixed by #3193
Closed

jsipfs config show throws error #3192

jacobheun opened this issue Jul 23, 2020 · 0 comments · Fixed by #3193
Labels
need/triage Needs initial labeling and prioritization

Comments

@jacobheun
Copy link
Contributor

  • Version: 0.48.1
  • Platform: Node.js v14.5.0
  • Subsystem: cli

Severity:

High

Description:

See steps to reproduce.

Steps to reproduce the error:

  • Install ipfs globally npm i -g ipfs
  • (Init if needed) jsipfs init
  • jsipfs config show
$ jsipfs config show
Cannot find module 'execa'
Require stack:
- /.nvm/versions/node/v14.5.0/lib/node_modules/ipfs/src/cli/commands/config/edit.js
- /.nvm/versions/node/v14.5.0/lib/node_modules/ipfs/node_modules/yargs/build/lib/yargs.js
- /.nvm/versions/node/v14.5.0/lib/node_modules/ipfs/node_modules/yargs/yargs.js
- /.nvm/versions/node/v14.5.0/lib/node_modules/ipfs/src/cli/parser.js
- /.nvm/versions/node/v14.5.0/lib/node_modules/ipfs/src/cli/index.js
- /.nvm/versions/node/v14.5.0/lib/node_modules/ipfs/src/cli/bin.js
@jacobheun jacobheun added the need/triage Needs initial labeling and prioritization label Jul 23, 2020
@jacobheun jacobheun mentioned this issue Jul 23, 2020
Merged
achingbrain pushed a commit that referenced this issue Jul 23, 2020
@jacobheun
fix: make execa a dep, it's used in ipfs config edit (#3193)
19b8113 
`execa` was being used a devDep but is required by the CLI for opening an editor during `ipfs config edit`. However, due to the way the cli requires modules, this also breaks other config commands.


fixes: #3192
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
need/triage Needs initial labeling and prioritization
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: make execa a dep, it's used in ipfs config edit ipfs/js-ipfs
1 participant
@jacobheun