Skip to content

Experimental snapshot sdk#49

Open
nicoche wants to merge 3 commits into
mainfrom
experimental-snapshot-sdk
Open

Experimental snapshot sdk#49
nicoche wants to merge 3 commits into
mainfrom
experimental-snapshot-sdk

Conversation

@nicoche

@nicoche nicoche commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@nicoche
nicoche force-pushed the experimental-snapshot-sdk branch from 5058f45 to ae7839d Compare July 2, 2026 09:33
@nicoche
nicoche force-pushed the experimental-snapshot-sdk branch from 4dfc860 to f7150e1 Compare July 20, 2026 15:08
nicoche and others added 2 commits July 20, 2026 17:27
- Remove duplicate imports in __init__.py
- Remove unused datetime import in sandbox.py
- Remove unused _from_api_snapshot method in snapshot.py
- Fix dead code in region extraction in _from_instance_api_snapshot
- Eliminate code duplication in _create_sync by building deployment definition once
- Improve error handling by catching SandboxError instead of broad Exception
- Remove redundant condition (snapshot_type is not None when already checked)

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
The InstanceSnapshot API object may not have a 'region' attribute.
Use hasattr() to safely check for region and regional_deployment_id.

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
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.

1 participant