Implement Copy Mode and Refactor Backup/Restore Operations #2085
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR introduces a new "copy mode" alongside the existing "link mode" for Mackup, providing users with two distinct approaches for managing their application settings. The version is bumped to 0.9.0 to reflect this significant feature addition.
Key changes:
- Refactored backup/restore operations into separate copy and link modes
- Renamed methods to clarify their functionality (e.g.,
backup()→link_install(),restore()→link(),uninstall()→link_uninstall()) - Updated documentation to explain both modes and removed outdated warnings
Reviewed Changes
Copilot reviewed 6 out of 7 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| pyproject.toml | Version bumped to 0.9.0 |
| mackup/main.py | Refactored command structure to support copy and link modes with new command syntax |
| mackup/application.py | Added new copy methods and renamed existing methods to reflect link mode functionality |
| mackup/mackup.py | Minor formatting change (added blank line) |
| doc/README.md | Fixed typo and added iCloud sync monitoring instructions |
| README.md | Comprehensive documentation update explaining copy vs link modes |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 6 out of 7 changed files in this pull request and generated 1 comment.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <[email protected]>
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 6 out of 7 changed files in this pull request and generated no new comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 7 out of 8 changed files in this pull request and generated 1 comment.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <[email protected]>
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 7 out of 8 changed files in this pull request and generated 5 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 9 out of 10 changed files in this pull request and generated no new comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
in PR lra/mackup#2085 now `mackup backup` uses flag `--force-no`
|
elegant solution to the Mac OS issue. Thank you! |
Adds "copy mode" as an alternative to "link mode" for managing app settings, enhancing flexibility. Refactors backup and restore operations to support both modes, renames methods (e.g.,
backup()tolink_install()), and updates README/docs.