Repository navigation
ci: remove staging SDK test jobs - #2040
devin-ai-integration[bot] wants to merge 2 commits into
Conversation
Co-Authored-By: mish@e2b.dev <mish@e2b.dev>
|
I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".
|
|
Co-Authored-By: mish@e2b.dev <mish@e2b.dev>
Package ArtifactsBuilt from d156f6f. Download artifacts from this workflow run. JS SDK ( npm install ./e2b-dockerfile-utils-0.1.2-devin-1793031989-remove-staging-tests.0.tgz ./e2b-2.61.1-devin-1793031989-remove-staging-tests.0.tgzCLI ( npm install ./e2b-cli-2.22.4-devin-1793031989-remove-staging-tests.0.tgzCode Interpreter JS SDK ( npm install ./e2b-code-interpreter-2.9.1-devin-1793031989-remove-staging-tests.0.tgzDesktop JS SDK ( npm install ./e2b-desktop-2.5.1-devin-1793031989-remove-staging-tests.0.tgzPython SDK ( pip install ./e2b_dockerfile_utils-0.1.1+devin.1793031989.remove.staging.tests-py3-none-any.whl ./e2b-2.61.0+devin.1793031989.remove.staging.tests-py3-none-any.whlCode Interpreter Python SDK ( pip install ./e2b_code_interpreter-2.11.5+devin.1793031989.remove.staging.tests-py3-none-any.whlDesktop Python SDK ( pip install ./e2b_desktop-2.7.2+devin.1793031989.remove.staging.tests-py3-none-any.whl |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Summary
Removes the seven
Staging / *jobs fromsdk_tests.yml(they ran againstvars.E2B_DOMAIN_STAGINGwithE2B_API_KEY_STAGING). They weren't part of the requiredSDK Tests Statuscheck, so PRs could already merge when they failed.Also removes the matrix and test switches that only the staging callers set from the reusable test workflows (
release.ymldoesn't pass any inputs):node-only(JS SDK, Code Interpreter JS): the matrices are now static lists of all runtimesrun_recovery_tests(Code Interpreter JS/Python): the interrupt suites always runrun_tests(Desktop JS/Python): tests always runThe
E2B_DOMAINinput stays on all reusable workflows so another domain can still be targeted.The
Production /prefix is dropped from the remaining job names. The only required check from this workflow isSDK Tests Status, which keeps its name.actionlintandprettier --checkpass.Linear: SDK-406
Link to Devin session: https://app.devin.ai/sessions/2acf81694185462e9024297c584a5746
Open in Devin Desktop: https://app.devin.ai/desktop/session/2acf81694185462e9024297c584a5746?variant=devin
Requested by: @mishushakov