googleapis / googleapis/librarian
command: pass verbose flag as config instead of global
Open
priority: p3
type: cleanup
- Dominant language
- Go
- Stars
- 45
- Forks
- 62
- Avg merge
- 18h 32m
- Merged PRs (30d)
- 175
Description
command.Verbose is currently a global, which forces tests to manage shared state.
We should pass the verbose flag as a config to avoid global mutation, simplify tests, and make the command behavior easier to reason about.
Contributor guide
Research direction
Locate command.Verbose and the command configuration entry points, then read the tests that currently manage its shared state. Trace how the verbose flag is passed and verify that tests no longer require global mutation while command behavior remains unchanged.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- go
- Domain
- cli
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100