Skip to content

Conversation

@lukaszo
Copy link
Member

@lukaszo lukaszo commented Jan 2, 2026

No description provided.

@lukaszo lukaszo requested a review from Copilot January 2, 2026 13:10
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds support for updating lifecycle settings on sandboxes through a new update_life_cycle method. The implementation moves lifecycle parameters from query parameters to request body fields, aligning with API best practices.

Key Changes

  • Added update_life_cycle() method to the Sandbox class for programmatic lifecycle management
  • Refactored API to accept lifecycle settings in request body instead of query parameters
  • Extended ServiceListItem model to include lifecycle information

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
spec/openapi.json Removed lifecycle query parameters and added lifecycle field to ServiceListItem schema
koyeb/sandbox/utils.py Added DeploymentsApi client and updated return type signature
koyeb/sandbox/sandbox.py Implemented update_life_cycle method and updated API client calls
koyeb/api/models/service_list_item.py Added lifecycle field with serialization support
koyeb/api/docs/ServicesApi.md Updated documentation to remove lifecycle query parameters
koyeb/api/docs/ServiceListItem.md Documented new lifecycle field
koyeb/api/api/services_api.py Removed lifecycle query parameters from update_service methods

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@lukaszo lukaszo merged commit 04e2b45 into main Jan 2, 2026
10 checks passed
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.

2 participants