-
-
Notifications
You must be signed in to change notification settings - Fork 824
✨ Support multiple options with multiple values #459
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
base: master
Are you sure you want to change the base?
Conversation
|
Now that I've opened it, I realize this is a possible duplicate of #362. I guess we must now battle for supremacy. |
This comment was marked as spam.
This comment was marked as spam.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
|
Hi @tiangolo, I wanted to inquire about the latest status on this PR. This PR has been pending for such a long time. I think this feature would be very useful. |
|
bump? :) |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
|
📝 Docs preview for commit db56796 at: https://54b33d2c.typertiangolo.pages.dev Modified Pages |
YuriiMotov
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@nealian, thanks for working on this!
Looks good in general!
Please, take a look at my suggestion
📝 Docs previewLast commit df934e8 at: https://a0c86031.typertiangolo.pages.dev Modified Pages |
This comment was marked as resolved.
This comment was marked as resolved.
YuriiMotov
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM in general!
@nealian, could you please address my comments?
'Cuz sometimes you gotta
--set-option-for object name value.