Skip to content

Conversation

@kellerben
Copy link
Contributor

Description

Added completions for btrbk

TODOs:

  • Changes to fish usage are reflected in user documentation/manpages.
  • Tests have been added for regressions fixed
  • User-visible changes noted in CHANGELOG.rst

Copy link
Contributor

@EmilyGraceSeville7cf EmilyGraceSeville7cf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please tell me whether you were satisfied with my review. 😄

end

# options with arguments
complete -c btrbk -l format -x -d 'Change output format' -a "table long raw"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AFAIS, --format is allowed just before any subcommand. I would rather add -n flag to complete to restrict when --format is shown, as well as other options falling in this category.

My comment is based on this man page:

       btrbk [-h|--help] [--version]
             [-c|--config <file>] [-n|--dry-run] [--exclude <filter>]
             [-p|--preserve] [--preserve-snapshots] [--preserve-backups]
             [-v|--verbose] [-q|--quiet] [-l|--loglevel <level>]
             [-t|--table] [-L|--long] [-1|--single-column]
             [--format <output-format>] [--pretty]
             [-S|--print-schedule] [--progress]
             [--lockfile <file>]
             [--override <config_option>=<value>]
             <command> [[--] <filter>...]

@faho faho merged commit 2ac1523 into fish-shell:master Dec 29, 2024
@faho
Copy link
Member

faho commented Dec 29, 2024

Thanks!

@faho faho added this to the fish 4.1 milestone Dec 29, 2024
@zanchey zanchey modified the milestones: fish 4.1, fish 4.0 Jan 2, 2025
zanchey pushed a commit that referenced this pull request Jan 2, 2025
* add completion for btrbk

completions for btrbk https://github.com/digint/btrbk/

* change indent + spaces

(cherry picked from commit 2ac1523)
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.

4 participants