Add CI/CD pipelines and Homebrew distribution#23
Merged
Conversation
Installing required cloning the repo and having Rust installed. No CI meant PRs landed without automated checks. Added GitHub Actions for CI (fmt, clippy, tests on all branches), cross-platform release builds (Linux, macOS, Windows), automatic beta builds from main, crates.io publishing, and a Homebrew tap workflow that pushes pre-built binaries to avelino/homebrew-mcp. Updated install docs to show brew and binary options first. fixed: #19 fixed: #17 Signed-off-by: Avelino <[email protected]>
Every command dumped raw JSON, which is great for scripts but terrible for humans typing in a terminal. Output now auto-detects context: interactive terminal gets colored tables (comfy-table + console) and a spinner during async operations (indicatif), piped/redirected output stays JSON for backwards compatibility. A --json flag forces JSON in any context. Tests go from 55 to 77 covering all format paths, content types and edge cases. CLI and scripting docs updated. fixed: #21 Signed-off-by: Avelino <[email protected]>
Error output was split across stdout/stderr for non-text content, spinner could leak terminal artifacts on early returns, and search emitted no JSON for empty results. Route all error content to stderr via an out! macro, add Drop impl and hidden-when-not-TTY to Spinner, always delegate to print_search_results regardless of result count, and revert Sentry URL to /sse for doc consistency. Signed-off-by: Avelino <[email protected]>
Signed-off-by: Avelino <[email protected]>
* origin/main: Fix review issues in output formatting PR Add human-friendly default output with auto-detection Signed-off-by: Avelino <[email protected]>
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
1 participant
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.
Installing required cloning the repo and having Rust installed. No CI meant PRs landed without automated checks.
Added GitHub Actions for CI (fmt, clippy, tests on all branches), cross-platform release builds (Linux, macOS, Windows), automatic beta builds from main, crates.io publishing, and a Homebrew tap workflow that pushes pre-built binaries to avelino/homebrew-mcp. Updated install docs to show brew and binary options first.
fixed: #19
fixed: #17