-
Notifications
You must be signed in to change notification settings - Fork 860
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 command unset #1475
add command unset #1475
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.
Thanks for your contribution! I've only had a few minutes so I only looked at the cmd/sops/main.go changes so far, I'll take a closer look at everything later.
213e477
to
6743394
Compare
The test failure was due to using the same file |
@felixfontein Any update on this? Would love this functionality. |
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.
Looks good from my side!
@duthils Good to merge? I'm planning on using this functionality shortly. Thanks for the change! |
@HarshSheth1128 Yes, good to merge, I have rebased the branch to avoid conflicts upon merge. |
Signed-off-by: Sebastien Duthil <duthils@duthils.net>
Signed-off-by: Sebastien Duthil <duthils@duthils.net>
Signed-off-by: Sebastien Duthil <duthils@duthils.net>
4bf37e3
to
2d914d2
Compare
@duthils thanks for your contribution! |
Add an
unset
command to remove a key or item from the file.Fixes #1305.
I'm not familiar with Go or Rust, so I mainly adapted what was already present in the
set
command.