Skip to content

Conversation

@sid-rl
Copy link
Contributor

@sid-rl sid-rl commented Nov 12, 2025

devboxes.create() has a mounts parameter that is missing from devboxes.create_and_await_running(). this adds in that parameter, which is passed through to create().

also, this cleans up the default parameters so that we appropriately use omit/not_given as default parameters in our manual wrappers instead of NOT_GIVEN to match generated function signatures.

…g, and cleaned up NOT_GIVEN default params to match generated code signatures
@sid-rl sid-rl requested review from alb-rl and tode-rl November 12, 2025 01:35
file_mounts: Optional[Dict[str, str]] | Omit = omit,
launch_parameters: Optional[LaunchParameters] | Omit = omit,
metadata: Optional[Dict[str, str]] | Omit = omit,
mounts: Optional[Iterable[Mount]] | Omit = omit,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this is the only extra parameter

file_mounts: Optional[Dict[str, str]] | Omit = omit,
launch_parameters: Optional[LaunchParameters] | Omit = omit,
metadata: Optional[Dict[str, str]] | Omit = omit,
mounts: Optional[Iterable[Mount]] | Omit = omit,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this is the only extra parameter

Copy link
Contributor

@alb-rl alb-rl left a comment

Choose a reason for hiding this comment

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

🚀

@sid-rl sid-rl merged commit 6ef7b34 into main Nov 12, 2025
7 checks passed
@sid-rl sid-rl deleted the siddarth/add-devbox-system-mounts branch November 12, 2025 01:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants