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 completions for yadm#7100

Merged
krobelus merged 3 commits into
fish-shell:masterfish-shell/fish-shell:masterfrom
sudormrfbin:add-yadm-completionssudormrfbin/fish-shell:add-yadm-completionsCopy head branch name to clipboard
Jun 30, 2020
Merged

Add completions for yadm#7100
krobelus merged 3 commits into
fish-shell:masterfish-shell/fish-shell:masterfrom
sudormrfbin:add-yadm-completionssudormrfbin/fish-shell:add-yadm-completionsCopy head branch name to clipboard

Conversation

@sudormrfbin

@sudormrfbin sudormrfbin commented Jun 9, 2020

Copy link
Copy Markdown
Contributor

Description

Add completions for yadm. It's wraps around git and all of git's features are usable.

TODOs:

  • User-visible changes noted in CHANGELOG.rst
  • Suppress git completions for clone subcommand as yadm has clone with different options
  • yadm gitconfig should inherit completions from git config
  • Respect --yadm-repo option

I'm not sure if suppressing inherited completions for specific options is possible or not, so could use a pointer.

Comment thread share/completions/yadm.fish Outdated
@sudormrfbin
sudormrfbin force-pushed the add-yadm-completions branch from 8436f19 to c4d1b2c Compare June 19, 2020 05:29
@sudormrfbin

Copy link
Copy Markdown
Contributor Author

The completions have been finished and now react to --yadm-repo option. It also correctly inherits (only) needed completions from git.

Comment thread share/completions/yadm.fish Outdated

@krobelus krobelus left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

looks good!

Comment thread share/completions/yadm.fish Outdated
# See `yadm gitconfig` for setting git config values
function __fish_yadm_config_keys
yadm introspect configs
end

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I guess this function could now be inlined, whatever you prefer.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I think I'll leave it as is, because __fish_yadm_config_keys seems more readable and intuitive to me than yadm introspect configs

@sudormrfbin
sudormrfbin force-pushed the add-yadm-completions branch from cd98827 to 3cf3433 Compare June 21, 2020 13:35
@krobelus
krobelus merged commit 3f210ac into fish-shell:master Jun 30, 2020
@krobelus krobelus added this to the fish 3.2.0 milestone Jun 30, 2020
@krobelus

Copy link
Copy Markdown
Contributor

Thanks and sorry for the delay!

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Sep 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

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