Skip to content

docs: require GitHub CLI auth for all install methods#25

Closed
christianalfoni wants to merge 1 commit intomainfrom
CSB-1267
Closed

docs: require GitHub CLI auth for all install methods#25
christianalfoni wants to merge 1 commit intomainfrom
CSB-1267

Conversation

@christianalfoni
Copy link
Copy Markdown
Collaborator

Add GITHUB_TOKEN check and Bearer auth header to install.sh so the binary download works on private repos. Update README install commands for CLI, TypeScript SDK, and Python SDK to use gh auth token instead of unauthenticated URLs.

Add GITHUB_TOKEN check and Bearer auth header to install.sh so the
binary download works on private repos. Update README install commands
for CLI, TypeScript SDK, and Python SDK to use `gh auth token` instead
of unauthenticated URLs.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@linear
Copy link
Copy Markdown

linear bot commented Apr 7, 2026

CSB-1267 Update install process to take advantage of GITHUB_TOKEN and GH CLI

CLI

So users can install using:

GITHUB_TOKEN=$(gh auth token) curl -fsSL -H "Authorization: Bearer $(gh auth token)" https://raw.githubusercontent.com/togethercomputer/together-sandbox/main/install.sh | bash

Update install.sh to put GITHUB_TOKEN into bearer of requests, if it is there

TypeScript

gh release download --repo togethercomputer/together-sandbox --pattern "together-sandbox-sdk.tgz" -D /tmp && npm install /tmp/together-sandbox-sdk.tgz

Python

pip install "together-sandbox @ git+https://$(gh auth token)@github.com/togethercomputer/together-sandbox.git#subdirectory=together-sandbox-python"

@christianalfoni christianalfoni marked this pull request as draft April 8, 2026 07:57
@christianalfoni
Copy link
Copy Markdown
Collaborator Author

Evaluating rather making the repo public

@christianalfoni
Copy link
Copy Markdown
Collaborator Author

We are rather going public repo

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.

1 participant