liquid-labs / liquid-labs/bash-rollup

Support multiple files

Open
#10 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

breaking enhancement
Dominant language
Shell
Stars
0
Forks
2
PR merge metrics
No merged PRs in 30d

Description

## Overview

Why limit ourselves to a single index file? Note, this is a breaking change

## Deliverables

* [ ] Process all positional arguments as files to be included (concatenated) one after the other.
* [ ] Update `usage.sh`.
* [ ] Use `bash-rollup.sh --help` to print usage and update `README.md` with changes. Note that `README.md` is markdown formatted, so the raw text may vary to achieve the appropriate formatting. Also, make any links live.

## Notes

The 'unique' inclusion rule should apply to all files. E.g., if file A and B both include `foo.sh`, then the contents will be inlined at A, but not B (unless flagged with `bash-rollup-always-inline`).

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the positional-argument handling in bash-rollup.sh, then read usage.sh and the current README.md instructions. Done means all positional files are concatenated with the stated unique-inclusion behavior, --help reflects the usage, and the README documents the change with live links.

Written by the indexing model from the issue text.

Assessment

Tech stack
bash
Domain
cli, documentation
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.