Skip to content

seamless sandboxing (just set GEMINI_CODE_SANDBOX=true in .env) - #76

Merged
olcan merged 1 commit into
mainfrom
seamless_sandbox
Apr 21, 2025
Merged

seamless sandboxing (just set GEMINI_CODE_SANDBOX=true in .env)#76
olcan merged 1 commit into
mainfrom
seamless_sandbox

Conversation

@olcan

@olcan olcan commented Apr 21, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@NTaylorMullen NTaylorMullen left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Solid. We may also want to write some dev docs for sandboxing as well

Comment thread scripts/build.sh

# also build container image if GEMINI_CODE_SANDBOX is set (can be in .env file)
# skip (-s) npm install + build since we did that above
if [[ "${GEMINI_CODE_SANDBOX:-}" =~ ^(1|true)$ ]] || grep -qiE '^GEMINI_CODE_SANDBOX *= *(1|true)' .env; then

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

AH I like this!

@olcan
olcan merged commit 39bdeda into main Apr 21, 2025
@olcan
olcan deleted the seamless_sandbox branch April 21, 2025 14:50
@olcan

olcan commented Apr 21, 2025

Copy link
Copy Markdown
Contributor Author

Solid. We may also want to write some dev docs for sandboxing as well

yes adding a Sandboxing section to the readme

yaleh pushed a commit to yaleh/gemini-cli that referenced this pull request Jun 29, 2025
viridivn pushed a commit to viridivn/gemini-cli that referenced this pull request Jun 29, 2025
@sripasg sripasg added the size/m A medium sized PR label Jun 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/m A medium sized PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants