Skip to content

Conversation

@jiasli
Copy link
Contributor

@jiasli jiasli commented Jan 10, 2020

Fix #67 (comment)

Add -o bashdefault to register-python-argcomplete's output command, so that individual command can complete the environment variables.

@codecov-io
Copy link

codecov-io commented Jan 10, 2020

Codecov Report

Merging #284 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #284   +/-   ##
=======================================
  Coverage   82.69%   82.69%           
=======================================
  Files           7        7           
  Lines         734      734           
=======================================
  Hits          607      607           
  Misses        127      127
Impacted Files Coverage Δ
argcomplete/shell_integration.py 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update abfa187...a45948e. Read the comment docs.


if complete_arguments is None:
complete_options = '-o nospace -o default' if use_defaults else '-o nospace'
complete_options = '-o nospace -o default -o bashdefault' if use_defaults else '-o nospace'
Copy link
Collaborator

Choose a reason for hiding this comment

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

We should also include -o bashdefault when use_defaults is False to match the behaviour of the global completion script.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Nice catch. Added as requested.

@evanunderscore
Copy link
Collaborator

Perfect, thanks!

@evanunderscore evanunderscore merged commit 7c314bc into kislyuk:master Jan 13, 2020
@kislyuk
Copy link
Owner

kislyuk commented Jan 14, 2020

Released in v1.11.1, please test.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

~/.bash_completion.d/python-argcomplete.sh prevents other completions

4 participants