feat: add lc formula - #1
Merged
bougyman merged 1 commit intoAug 11, 2026
Merged
Conversation
The formula was pointing at v0.8.1 assets/sha256 - stale placeholders from before a real release existed. Points both URLs at the actual v1.0.0 release and updates sha256 to match the release's own SHA256SUMS file. Verified: downloaded both tarballs, confirmed a flat layout (lc + the 5 wrapper scripts at tarball root, matching bin.install's arg list), and ran the extracted linux_x86_64 binary's `--version` output directly.
There was a problem hiding this comment.
Pull request overview
Updates the Homebrew tap formula for lc to install the v1.0.0 precompiled release assets, aligning the formula’s URLs and SHA256 checksums with the latest upstream release.
Changes:
- Bump macOS (arm64) download URL and SHA256 to
v1.0.0. - Bump Linux (x86_64) download URL and SHA256 to
v1.0.0.
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
bougyman
deleted the
bougyman/cry-37-create-rubyistshomebrew-tap-for-lc
branch
August 11, 2026 21:28
bougyman
restored the
bougyman/cry-37-create-rubyistshomebrew-tap-for-lc
branch
August 12, 2026 12:14
Merged
5 tasks
bougyman
added a commit
to rubyists/linear-cli
that referenced
this pull request
Aug 12, 2026
#75) ## Summary - Vendors the new `rubyists/homebrew-tap` repo as a submodule at `vendor/rubyists-homebrew-tap`, pinned to the commit whose `lc` formula points at the real `v1.0.0` release assets/sha256 (rubyists/homebrew-tap#1). Closes CRY-37. - Rewrites `install.sh` to be actually curl-pipeable: it previously required a local repo checkout (`BASH_SOURCE[0]`-derived `app/` path) and a full Elixir/Erlang/Zig toolchain to build from source, despite its own comment claiming to be the Homebrew-less fallback. Now downloads the release tarball for the detected platform (already bundles the wrapper scripts), verifies it against the release's own `SHA256SUMS`, and installs - no toolchain needed. Adds `LC_VERSION` to pin a specific release tag. Also strips macOS's quarantine attribute on `lc` (previously a manual README step). - Fixes a real bug found while testing: a checksum-verification failure didn't actually stop the script (a missing `|| exit 1` on the one command-substitution call that lacked it), so it fell through into an unrelated second error instead of failing clean. - Updates the README: fixes the stale "Homebrew tap is planned but not yet available" note now that the tap exists, adds a Homebrew install subsection, and documents `install.sh`/`uninstall.sh`'s new curl-pipeable one-liners. ## Test plan - [x] Ran `install.sh` directly and piped (`cat install.sh | bash`, matching real `curl | bash` usage) - both install correctly, `lc --version` reports `1.0.0` - [x] Tested `LC_VERSION=v1.0.0` (pinned) alongside default (`latest`) - [x] Tested a corrupted/tampered download - checksum verification correctly aborts before installing anything - [x] Round-tripped with `uninstall.sh` - removes exactly the manifest's files - [x] Verified the Homebrew formula's sha256 values against the release's own `SHA256SUMS`, and the tarball's flat layout against `bin.install`'s arg list 🤖 Generated with [Claude Code](https://claude.com/claude-code)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
lc(the Elixir port oflinear-cli, https://github.com/rubyists/linear-cli-ex), installing a precompiled binary for macOS (Apple Silicon) and Linux (x86_64) - no Elixir/Erlang/Zig build dependency needed on the installing machine.v1.0.0release assets/sha256 (matching that release's ownSHA256SUMS) - verified by downloading both tarballs, confirming the flat layout (lc+ the 5 wrapper scripts at tarball root), and running the extractedlinux_x86_64binary's--versionoutput directly.Closes CRY-37.
Test plan
ruby -c Formula/lc.rb- syntax OKSHA256SUMSassetbin.install's arg list (flat, no subdirectory)brew install --build-from-source rubyists/tap/lcon an actual macOS/Linux machine (nobrewavailable in this environment)🤖 Generated with Claude Code