liquid-labs / liquid-labs/bash-rollup
Make 'output' optional
Nobody has claimed this yet.
- Dominant language
- Shell
- Stars
- 0
- Forks
- 2
- PR merge metrics
- No merged PRs in 30d
Description
## Overview
Let's make the output arg optional. If not present, spit results to `stdout`. This is useful for something like `eval $(bash-rollup index.sh)` or `source <(bash-rollup index.sh)`
## Deliverables
* [ ] Add `--output|-o` option and set the output file based on this.
* [ ] For now, keep support of output as second argument, but emit a warning: `WARNING: support specifying file output as the second argument will be removed in a future release. Use the `--output` option.`
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the bash-rollup command entry point and inspect how its positional output argument is parsed; the issue names no file or test. Done means output is optional and defaults to stdout, --output/-o selects a file, and the legacy second argument still works while emitting the specified warning.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- shell
- Domain
- cli
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 52/100