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

Fix ZSH completion script #12173

Merged
merged 1 commit into from
Oct 7, 2023
Merged

Fix ZSH completion script #12173

merged 1 commit into from
Oct 7, 2023

Conversation

Freed-Wu
Copy link
Contributor

No description provided.

@uranusjr uranusjr changed the title Fix #12166 Fix ZSH completion script Jul 23, 2023
@henry0312
Copy link

Is there any new progress?

@sbidoul
Copy link
Member

sbidoul commented Oct 1, 2023

@Freed-Wu can you check why CI is red?

@dtrifiro
Copy link

dtrifiro commented Oct 6, 2023

I fixed the broken test in my fork, and added news entry as well (which should fix CI).

The test was failing because of a wrong indentation of the expected result. See
23e1c8b (main...dtrifiro:pip:fix-zsh-completion) for the fix.

I can also confirm that the this PR does fix completion, thanks!

Looking forward to seeing this merged 🚀

- tests expected results indendation was off
- add bugfix news entry
@Freed-Wu
Copy link
Contributor Author

Freed-Wu commented Oct 7, 2023

@dtrifiro I rename your bugfix to bugfix.rst. Now all CIs have passed. Can @pradyunsg review the PR again or merge it?

@pradyunsg pradyunsg merged commit 9b73482 into pypa:main Oct 7, 2023
@pradyunsg
Copy link
Member

Merging based on the user confirmation that this works well! :)

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants