Joystream / Joystream/joystream

CLI: Don't `exit()`, accept `--yes` and array inputs for batch operations

Open
#5,224 0 comments 0 reactions 0 assignees View on GitHub
CLI
Dominant language
Rust
Stars
1.4k
Forks
116
PR merge metrics
No merged PRs in 30d

Description

Context: #1571, #1591

## Scope

- The CLI uses [exit()](https://github.com/Joystream/joystream/blob/master/cli/src/ExitCodes.ts) and cannot be called from node to manage multiple tx after another or in parallel because the child process exits the parent.
- Many commands require using `yes | joystream-cli` to force the cli to work. Add `--yes` as option to all commands that lack it.
- Managing content often requires calling the same function hundreds of times. Offer to submit parameters as array whenever possible / it makes sense based on context. For example accept `assetId` / `videoId` pairs for `content:deleteChannelAssetsAsModerator`, or even accept a list of `assetId` and fetch the according video internally.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.