Skip to content

fix: resolve remaining merged-PR review follow-ups#121

Merged
rmanibus merged 1 commit into
mainfrom
fix/legacy-pr-review-followups
Mar 16, 2026
Merged

fix: resolve remaining merged-PR review follow-ups#121
rmanibus merged 1 commit into
mainfrom
fix/legacy-pr-review-followups

Conversation

@rmanibus
Copy link
Copy Markdown
Contributor

Summary

  • apply still-relevant follow-ups from merged PR review comments across #61, #72, #74, #83, #84, and #86
  • harden source behavior for edge cases: incremental delete parent fallback, Google Drive nil change payloads, root-relative path handling, OneDrive root-path normalization/encoding, and cleanup of accidental test artifact files
  • tighten CLI/profile/completion behavior (store URI validation, boolean flag completion parsing, deep-clone missing global flags), fix pack-store logging race, and align benchmark/check scripts with current password/env behavior

Validation

  • go test -count=1 ./pkg/source
  • go test -count=1 ./cmd/cloudstic
  • go test -count=1 ./...
  • golangci-lint run ./...

@rmanibus rmanibus merged commit 4d4c840 into main Mar 16, 2026
5 checks passed
@rmanibus rmanibus deleted the fix/legacy-pr-review-followups branch March 16, 2026 09:07
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 16, 2026

@rmanibus rmanibus added the chore Maintenance and tooling changes label Mar 16, 2026
@rmanibus rmanibus added this to the v1.12.0 milestone Mar 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Maintenance and tooling changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant