Conversation
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ec0faa786c
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
ValentaTomas
left a comment
There was a problem hiding this comment.
Added comment, but lgtm.
|
@ValentaTomas need your approval |
Package ArtifactsBuilt from d0efb08. Download artifacts from this workflow run. JS SDK ( npm install ./e2b-2.12.2-always-enable-keep-alive.0.tgzCLI ( npm install ./e2b-cli-2.7.1-always-enable-keep-alive.0.tgzPython SDK ( pip install ./e2b-2.13.2+always.enable.keep.alive-py3-none-any.whl |
Note
Low Risk
Comment-only change; no runtime behavior, API, or data-handling logic is modified.
Overview
Adds an inline note in the JS SDK
ApiClientandEnvdApiClientcreateClientconfiguration clarifying that HTTP/1.1 connections are persistent by default.Keeps the
keepaliveoption commented out (removing the prior TODO wording) to avoid implying an active behavior change while documenting the intent.Written by Cursor Bugbot for commit 5572ffb. This will update automatically on new commits. Configure here.