Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Add --recursive flag#1598

Open
qiu-x wants to merge 1 commit into
getsops:maingetsops/sops:mainfrom
qiu-x:mainqiu-x/sops:mainCopy head branch name to clipboard
Open

Add --recursive flag#1598
qiu-x wants to merge 1 commit into
getsops:maingetsops/sops:mainfrom
qiu-x:mainqiu-x/sops:mainCopy head branch name to clipboard

Conversation

@qiu-x

@qiu-x qiu-x commented Sep 7, 2024

Copy link
Copy Markdown

Adds a --recursive flag like described in #1204.

@qiu-x

qiu-x commented Sep 7, 2024

Copy link
Copy Markdown
Author

@schlichtanders please let me know if this works as expected for you

@qiu-x
qiu-x force-pushed the main branch 4 times, most recently from edbcff5 to 57ab01c Compare September 7, 2024 08:35
Signed-off-by: Aleksander Słomka <alex@alexslomka.xyz>
@felixfontein

Copy link
Copy Markdown
Contributor

Thanks for your contribution! I don't think this is a good idea though. We don't want to add new features to the --encrypt / --decrypt / ... flags, but instead concentrate on the encrypt, decrypt, edit, ... subcommands (see also #1333).

I'm also not sure whether it's a good idea to add this to the encrypt and decrypt subcommands, it might be better to have separate subcommands for this, since the is sufficiently different from regular encryption/decryption (you don't need a specific filename, but rather an optional path; you always encrypt/decrypt in-place; etc.).

@qiu-x

qiu-x commented Sep 9, 2024

Copy link
Copy Markdown
Author

Thanks for your contribution! I don't think this is a good idea though. We don't want to add new features to the --encrypt / --decrypt / ... flags, but instead concentrate on the encrypt, decrypt, edit, ... subcommands (see also #1333).

I see, I have been wondering why there are two ways of executing actions, but #1333 explains everything.

I'm also not sure whether it's a good idea to add this to the encrypt and decrypt subcommands, it might be better to have separate subcommands for this, since the is sufficiently different from regular encryption/decryption (you don't need a specific filename, but rather an optional path; you always encrypt/decrypt in-place; etc.).

Personally, I am more in favor of a --recursive flag, because that seems analogous to the interface of other CLI tools. Currently, this change also works with modes like --rotate or --unset, so adding subcommands instead of a flag, would lead to a lot of duplication from my perspective.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.