-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Support multiple values in native completions #5601
Conversation
I can pass the |
Thank you for your review! Sorry for the late response. I have made the changes to the code. During the modification process, I discovered a small error. For
The original incorrect logic would treat pos_1 as the first value of the positional argument with index=2, leading to a count error for index=2. Therefore, I changed |
If we've identified a bug like this during development, it means we should probably add a test case for it. |
…ument with `num_args(N)`
Related issue #3921
Work in this PR
What we can complete now?
bar1 bar2 bar3
.Also,
bar1 bar2 bar3
.num_args(N)
which N is fixed.pos_a pos_b pos_c
pos_a pos_b pos_c
What may be is left
last
ortrailing_var_arg