-
Notifications
You must be signed in to change notification settings - Fork 634
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add bbolt surgery revert-meta-page
command
#385
Add bbolt surgery revert-meta-page
command
#385
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you.
|
||
// RevertMetaPageCommand represents the "surgery revert-meta-page" command execution. | ||
type RevertMetaPageCommand struct { | ||
baseCommand |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that all surgery commands should operate on SrcPath and DstPath.
I think with the next tool the code will be better structured if we have composition with
a common tool: baseSurgeryToolCommand
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let me do this when I add the next surgery command.
Signed-off-by: Benjamin Wang <wachao@vmware.com>
Just rebased this PR and address review comments. Let's consider to add more unit test later in separate PR. |
Signed-off-by: Benjamin Wang <wachao@vmware.com>
Thanks @ptabor for the quick review! |
Linked to #370
Note that the first commit is based on #384. Once 384 is merged, I will rebase this PR.