release v0.0.1 - #27
Conversation
Signed-off-by: kerthcet <kerthcet@gmail.com>
There was a problem hiding this comment.
🟡 Not ready to approve
The installer uses a non-portable mktemp invocation that will fail on macOS, and the README’s pinned-version example doesn’t match the v0.0.1 release.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
This review doesn't count toward merge requirements. Sign up for the private preview to control whether Copilot approvals count.
Pull request overview
This PR prepares the repository for the v0.0.1 release by bumping package versions, switching the daemon’s WebSocket TLS stack to rustls for musl-friendly static binaries, and adding a GitHub Actions workflow to build and publish release artifacts.
Changes:
- Bump Rust crate and Python package versions from
0.0.0to0.0.1. - Switch
tokio-tungsteniteto rustls-based TLS to avoid OpenSSL and better support static musl builds. - Add a tag-triggered release workflow and update installation docs + installer to download GitHub release assets (with cargo fallback).
File summaries
| File | Description |
|---|---|
| sandd/Cargo.toml | Bumps daemon version and switches tokio-tungstenite to rustls TLS features. |
| pyproject.toml | Bumps Python package version to match the release. |
| Cargo.lock | Updates lockfile to reflect rustls/ring dependency changes and version bump. |
| hack/scripts/install.sh | Installs from GitHub release assets first, adds platform-aware asset naming, and improves cargo fallback behavior. |
| README.md | Updates install instructions to use the repo-hosted install script and documents version pinning / git install. |
| .github/workflows/release.yaml | Adds release automation for building per-platform assets and publishing a GitHub Release with checksums. |
Review details
- Files reviewed: 5/6 changed files
- Comments generated: 2
- Review effort level: Lite
We're testing this review assessment. Please use 👍 or 👎 to tell us if it's correct.
|
/lgtm |
InftyAI-Agent
left a comment
There was a problem hiding this comment.
Approved: PR has both lgtm and approved labels
What this PR does / why we need it
Which issue(s) this PR fixes
Fixes #
Special notes for your reviewer
Does this PR introduce a user-facing change?