-
Notifications
You must be signed in to change notification settings - Fork 874
[Jobs][CLI] allow unknown options in jobs cli #3614
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
Conversation
|
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update. |
|
We've recently got this PR to add a I think we should stick with this syntax that feels cleaner and explicitly tells that the rest of the command is an argument. WDYT ? |
|
if we want to stick to the UV and Docker like CLI experience I think we should support commands without having to add (note that using |
|
wdyt ? cc @davanstrien for viz as well |
|
No strong opinion. I also ran into this a few times. Maybe it's still a bit cleaner to rely on I found in general, coding agents (at least Claude) seemed to fairly quickly grasp the It could make sense to use |
|
UV docs mention
and in their examples they mostly use without
Docker on their side never mention edit: I added a mention of |
Wauplin
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.
Thanks @lhoestq for this thread! Agree with you and @davanstrien that supporting both while only mentioning -- is a correct option. I'll merge this now then :)
fix this error
and