Skip to content
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

[23.0 backport] Fix plugin completion parsing for plugins using ShellCompDirectiveFilterFileExt #4177

Merged

Conversation

thaJeztah
Copy link
Member

(cherry picked from commit 683e4bf)

- What I did

- How I did it

- How to verify it

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

…lterFileExt`

Signed-off-by: Laura Brehm <[email protected]>
(cherry picked from commit 683e4bf)
Signed-off-by: Sebastiaan van Stijn <[email protected]>
@codecov-commenter
Copy link

codecov-commenter commented Apr 9, 2023

Codecov Report

Merging #4177 (a763e51) into 23.0 (24d608d) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##             23.0    #4177   +/-   ##
=======================================
  Coverage   59.02%   59.02%           
=======================================
  Files         288      288           
  Lines       24810    24810           
=======================================
  Hits        14644    14644           
  Misses       9279     9279           
  Partials      887      887           

Copy link
Collaborator

@albers albers left a comment

Choose a reason for hiding this comment

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

LGTM.

@thaJeztah
Copy link
Member Author

thx! Let me bring this one in

@thaJeztah thaJeztah merged commit 726dfe9 into docker:23.0 Apr 11, 2023
@thaJeztah thaJeztah deleted the 23.0_backport_fix_bash_file_completion branch April 11, 2023 07:29
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