Clean up cmd/sops/main.go, get rid of constant
Open
Nobody has claimed this yet.
refactoring
- Dominant language
- Go
- Stars
- 23.1k
- Forks
- 1.1k
- Avg merge
- 1d 11h
- Merged PRs (30d)
- 9
Description
Streamline how the config is loaded (right now it is loaded up to four times in some situations, like when running sops edit foo.yml). (Ref: https://github.com/getsops/sops/pull/1820#pullrequestreview-2727929345)
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
In cmd/sops/main.go, trace the configuration-loading path used by commands such as sops edit foo.yml, starting with the review reference in pull request 1820. Confirm where the configuration is loaded repeatedly and identify the existing constant involved. Done means the command no longer loads the configuration up to four times in the described situation.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- go
- Domain
- cli
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100