Skip to content

Conversation

@adam-rl
Copy link
Contributor

@adam-rl adam-rl commented Nov 25, 2025

No description provided.

@adam-rl adam-rl requested review from sid-rl and tode-rl November 25, 2025 23:13
@adam-rl adam-rl force-pushed the adam/build-context-docs branch from 9e74cf6 to 5bee9b1 Compare November 25, 2025 23:39
Example:
>>> from datetime import timedelta
>>> from runloop_api_client.types.blueprint_build_parameters import BuildContext
Copy link
Contributor

Choose a reason for hiding this comment

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

From an ergonomics standpoint, this is pretty ugly. I don't think we should try to fix this now, but it would be nice to avoid needing to pollute the imports with a bunch of specific types.

For this specific case, what if the storage Object had a to_build_context()? Then we skip the import and just do

blueprint = await runloop.blueprint.create(
... name="my-blueprint",
... dockerfile="FROM ubuntu:22.04\nCOPY . .\n",
... build_context=obj.to_build_context(),
... )

@adam-rl adam-rl merged commit d8b7960 into next Nov 26, 2025
6 checks passed
@adam-rl adam-rl deleted the adam/build-context-docs branch November 26, 2025 00:07
@stainless-app stainless-app bot mentioned this pull request Nov 26, 2025
stainless-app bot added a commit that referenced this pull request Dec 2, 2025
* chore: hide build context APIs

* fix(devbox): launch parameter typo

* fix(scorer): fixed RL_TEST_CONTEXT to RL_SCORER_CONTEXT

* fix(api): don't ignore devbox keep_alive, suspend and resume in api

* feat(blueprints): Add build context to the OpenAPI spec (#6494)

* chore(mounts): Update documentation for deprecated fields to direct the user to the replacement API

* chore(blueprints): Add build context examples (#694)

* feat(sdk): added scorer classes to sdk (#698)

* added scorer class (kept create and list as static methods for now since we don't know how we're creating scorers yet)

* refactored static methods to ScorerOps class

* fix example docstrings to use correct scorer create params

* scorer tests

* fixed scorer unit test parameters for update and validate

* update scorer and scorer ops docstrings to be more helpful while not exposing system internals

* update docs with scorer classes, methods and types

* remove verbose request options in unit test parameters

* rename client to ops in client test

* rename client test file to ops

* added list_empty, list_single and list_multiple unit tests to all ops class tests

* fix assert_called to assert_awaited

* remove duplicate tests

* release: 1.0.0

---------

Co-authored-by: stainless-app[bot] <142633134+stainless-app[bot]@users.noreply.github.com>
Co-authored-by: Adam Lesinski <adam@runloop.ai>
Co-authored-by: sid-rl <siddarth@runloop.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.

4 participants