Abstract cli arg parser fully into the `args` package
Nobody has claimed this yet.
- Dominant language
- Go
- Stars
- 6
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Description
### Problem Statement
Our use of the `urfave/cli` package exists in a few different parts of our code base. Not that we plan to change it, but it's better practice to use abstractions to confine the dependency to smaller part of our code base.
### Proposed Solution
Add necessary abstractions to our `args` package so that it is the only place in our code where `urfave/cli` is imported and used.
### Exit Criteria
- [ ] `urfave/cli` is only imported in the `args` package
Contributor guide
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 by searching the codebase for imports and uses of urfave/cli, then inspect the args package and the CLI entry points that depend on them. Done means the args package provides the necessary abstractions and urfave/cli is imported only there; no tests or specific files are named in the issue.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- go
- Domain
- cli, tooling
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 50/100