Skip to content

Added an install script#2

Merged
3coins merged 3 commits into
mainfrom
install-script
Mar 13, 2026
Merged

Added an install script#2
3coins merged 3 commits into
mainfrom
install-script

Conversation

@3coins

@3coins 3coins commented Mar 13, 2026

Copy link
Copy Markdown
Collaborator

Adds an install script for users to install binaries directly from the release artifacts.

curl -fsSL https://raw.githubusercontent.com/jupyter-ai-contrib/nb-cli/main/install.sh | bash

This installs the binary to ~/.nb-cli/bin/nb. Follow the instructions to add it to your PATH.

@ellisonbg

Copy link
Copy Markdown
Collaborator

I think this is a good start until we build up the stats to start to use brew.

@3coins 3coins marked this pull request as ready for review March 13, 2026 05:42

@andrii-i andrii-i 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.

Thank you @3coins. Please see a bug in install.sh (see inline) with 1-char fix below. Everything else looks good.

Comment thread install.sh Outdated
Comment thread README.md

This installs the binary to `~/.nb-cli/bin/nb`. Follow the instructions to add it to your PATH.

**Note**: Pre-built binaries are currently only available for macOS Apple Silicon (M1/M2/M3/M4). For other platforms, please use `cargo install` or build from source.

@andrii-i andrii-i Mar 13, 2026

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.

(Note to bring attention, not a request for change)
This is accurate for v0.0.1 but prep-release.yml also builds macOS arm64/amd64, Linux amd64/arm64, Windows. So we should update this after next release.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Right, have to verify those actually work in the CI yet, will update README after that.

@andrii-i andrii-i 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.

Thank you @3coins.

uses: dtolnay/rust-toolchain@stable

- name: Publish to crates.io
run: cargo publish

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.

cargo publish with no CARGO_REGISTRY_TOKEN assumes trusted publishing is configured on crates.io for this repo. If not, this will fail. Worth double checking.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Correct, I will turn it on once the release.yml is merged.

Comment thread install.sh
URL="https://github.com/$REPO/releases/download/$VERSION/$BINARY"

# Installation directory (can be overridden with INSTALL_DIR env var)
INSTALL_DIR="${INSTALL_DIR:-$HOME/.nb-cli/bin}"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

non-blocking: this usually goes to usr/bin/<tool-name> (or sometimes to usr/local/bin/<tool-name>).

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Yes, that makes sense, I am not aware of platform specific variations, so went with an independent path for the binary. We can explore using one of these if you are confident on the paths.

@3coins 3coins merged commit acca911 into main Mar 13, 2026
1 check passed
@andrii-i andrii-i mentioned this pull request Apr 9, 2026
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