Skip to content

refactor: group backup source init options#156

Merged
rmanibus merged 1 commit into
mainfrom
refactor/init-source-options
Mar 18, 2026
Merged

refactor: group backup source init options#156
rmanibus merged 1 commit into
mainfrom
refactor/init-source-options

Conversation

@rmanibus
Copy link
Copy Markdown
Contributor

Closes #155

Summary

  • replace the long positional initSource(...) signature with an initSourceOptions struct
  • update backup and auth call sites to construct source initialization options explicitly
  • refresh the source-init tests to exercise the refactored API

Testing

  • go test ./cmd/cloudstic ./internal/engine ./pkg/source ./pkg/store
  • golangci-lint run ./cmd/cloudstic/... ./internal/engine/... ./pkg/source/... ./pkg/store/...

@rmanibus rmanibus force-pushed the refactor/init-source-options branch from 5d0f64f to ba6e6a5 Compare March 18, 2026 10:03
@rmanibus rmanibus added the chore Maintenance and tooling changes label Mar 18, 2026
@rmanibus rmanibus added this to the v1.14.0 milestone Mar 18, 2026
@rmanibus rmanibus merged commit eab5efa into main Mar 18, 2026
7 checks passed
@rmanibus rmanibus deleted the refactor/init-source-options branch March 18, 2026 10:07
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 18, 2026

Codecov Report

❌ Patch coverage is 55.17241% with 26 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
cmd/cloudstic/cmd_backup.go 69.56% 14 Missing ⚠️
cmd/cloudstic/cmd_auth.go 0.00% 12 Missing ⚠️

📢 Thoughts on this report? Let us know!

@rmanibus rmanibus added refactor and removed chore Maintenance and tooling changes labels Apr 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Refactor backup source initialization options

1 participant