docs(install): recommend global hooks as primary setup path by jdx · Pull Request #855 · jdx/hk

jdx · 2026-04-23T17:41:35Z

Summary

Leads the install docs with hk install --global as the recommended setup path. The global install is a silent no-op in repos without an hk.pkl, so it's safe to enable once per machine and avoids re-running hk install in every clone.

Follow-up to #853 (the config-based hooks + --global feature that this documents).

What changed

docs/getting_started.md — restructured: binary install → Install Hooks (recommended: global) → Project Setup → config example. Per-repo hk install is now framed as an alternative for pre-Git-2.54 or selective-repo use. The overlap warning (Git aggregates hook.<name>.command across scopes) and the manual ~/.gitconfig setup are retained as subsections.
src/cli/install.rs — rewrote the Install struct docstring and --global flag help so hk install --help leads with the recommendation.
docs/cli/install.md, hk.usage.kdl, docs/cli/commands.json — regenerated via hk usage → usage g markdown.
Renamed the pre-existing "Global Configuration" section to "Global `hkrc` Configuration" in getting_started.md to disambiguate from global hooks (the sections are adjacent and share the word "global").
docs/package.json — added "version": "0.0.0" + "private": true so the bun workspace resolves cleanly (the pre-push docs:build hook was failing on missing version).

Why

New users land on getting_started.md; leading with per-repo hk install made them re-install in every clone and gave no signal that the global option existed. The --from-hook short-circuit (also from #853) makes global genuinely safe to enable everywhere, so it deserves the headline.

Test plan

hk check --all passes.
Pre-push hook (runs full docs build) passes.
Review rendered getting_started and cli/install pages after merge.

🤖 Generated with Claude Code

gemini-code-assist

Code Review

This pull request implements global git hook support for hk using Git 2.54+ config-based hooks, adding --global, --legacy, and --from-hook flags to manage these hooks safely across repositories. Documentation and tests were updated to support these features. Review feedback identified several compilation issues due to missing info! macro imports and the use of unstable let_chains syntax, as well as an opportunity to optimize the removal of git configuration sections.

gemini-code-assist · 2026-04-23T17:44:16Z

@@ -1,15 +1,48 @@
 use crate::{Result, config::Config, env, git_util};
+use eyre::bail;
 use log::warn;


The info! macro is used later in this file (line 269) but is not imported. This will cause a compilation error unless the log macros are globally available via a crate-level #[macro_use]. Given that warn is explicitly imported, info should be as well for consistency and correctness.

Suggested change

use log::warn;

use log::{info, warn};

gemini-code-assist · 2026-04-23T17:44:16Z

+        if let Ok(output) = Command::new("git")
+            .args(["config", "--global", "--get", key.as_str()])
+            .output()
+            && output.status.success()
+            && !output.stdout.is_empty()
+        {
+            overlapping.push(event);
+        }


This code uses the let_chains feature (if let ... && ...), which is currently unstable in Rust (see tracking issue #53667). Unless this project specifically requires a nightly toolchain and has the feature enabled, this will cause a compilation error on stable Rust. It is safer to use nested if statements.

Suggested change

if let Ok(output) = Command::new("git")

.args(["config", "--global", "--get", key.as_str()])

.output()

&& output.status.success()

&& !output.stdout.is_empty()

{

overlapping.push(event);

}

if let Ok(output) = Command::new("git")

.args(["config", "--global", "--get", key.as_str()])

.output()

{

if output.status.success() && !output.stdout.is_empty() {

overlapping.push(event);

}

}

gemini-code-assist · 2026-04-23T17:44:16Z

@@ -1,33 +1,25 @@
-use crate::{Result, git_util};
+use crate::{Result, cli::install};


The info! macro is used in this file (lines 15 and 22) but is not imported. This will lead to a compilation error.

Suggested change

use crate::{Result, cli::install};

use crate::{Result, cli::install};

use log::info;

gemini-code-assist · 2026-04-23T17:44:16Z

+pub(crate) fn remove_config_entries(scope: &str) -> Result<usize> {
+    let output = Command::new("git")
+        .args([
+            "config",
+            scope,
+            "--name-only",
+            "--get-regexp",
+            "^hook\\.hk-",
+        ])
+        .output()?;
+    // git config --get-regexp: 0 = matches, 1 = no matches, ≥2 = real error
+    // (e.g. unreadable config). Don't conflate "nothing to remove" with a
+    // failed uninstall.
+    let code = output.status.code().unwrap_or(1);
+    if code == 1 {
+        return Ok(0);
+    }
+    if !output.status.success() {
+        bail!(
+            "git config --get-regexp failed (exit {}): {}",
+            code,
+            String::from_utf8_lossy(&output.stderr).trim()
+        );
+    }
+    let keys: Vec<String> = String::from_utf8_lossy(&output.stdout)
+        .lines()
+        .map(|s| s.trim().to_string())
+        .filter(|s| !s.is_empty())
+        .collect();
+    // Dedupe since multi-valued keys appear once per value.
+    let mut seen = std::collections::BTreeSet::new();
+    let mut removed = 0;
+    for key in keys {
+        if seen.insert(key.clone()) {
+            run_git(&["config", scope, "--unset-all", key.as_str()])?;
+            // Count one per hook event, not one per key (command + event).
+            if key.ends_with(".command") {
+                removed += 1;
            }
-            let hook_file = hooks.join(hook);
-            xx::file::write(&hook_file, git_hook_content(&command, hook))?;
-            xx::file::make_executable(&hook_file)?;
-            println!("Installed hk hook: {}", hook_file.display());
        }
-        Ok(())
    }
+    Ok(removed)
+}


The current implementation of remove_config_entries is inefficient as it spawns a new git config --unset-all process for every single key found. For a standard installation with 9 hooks, this results in 18 process spawns. Additionally, unsetting keys individually leaves empty section headers (e.g., [hook "hk-pre-commit"]) in the git config file.

A more efficient and cleaner approach is to identify the unique section names and use git config --remove-section, which removes all keys within the section and the section header itself in a single call per hook.

pub(crate) fn remove_config_entries(scope: &str) -> Result<usize> { let output = Command::new("git") .args(["config", scope, "--name-only", "--get-regexp", "^hook\\.hk-"]) .output()?; // git config --get-regexp: 0 = matches, 1 = no matches, ≥2 = real error // (e.g. unreadable config). Don't conflate "nothing to remove" with a // failed uninstall. let code = output.status.code().unwrap_or(1); if code == 1 { return Ok(0); } if !output.status.success() { bail!( "git config --get-regexp failed (exit {}): {}", code, String::from_utf8_lossy(&output.stderr).trim() ); } let stdout = String::from_utf8_lossy(&output.stdout); let mut sections = std::collections::BTreeSet::new(); for key in stdout.lines().filter(|s| !s.trim().is_empty()) { if let Some(section) = key.rsplitn(2, '.').nth(1) { sections.insert(section.to_string()); } } let removed = sections.len(); for section in sections { run_git(&["config", scope, "--remove-section", &section])?; } Ok(removed) }

greptile-apps · 2026-04-23T17:45:09Z

Greptile Summary

This PR promotes hk install --global as the recommended setup path, adds Git 2.54+ config-based hook installation at both global and local scopes, and restructures the docs accordingly. The logic in install.rs is well-structured — idempotent cleanup before re-install, correct exit-code handling in remove_config_entries, and a warn_if_global_overlap guard for double-fire detection.

One open concern: run_git (used for all git config writes) calls .status() and discards stderr, so failures like permission-denied or a locked config file surface only as \"git config … failed\" with no reason. remove_config_entries already uses .output() and includes stderr in the error — run_git should follow the same pattern.

Confidence Score: 4/5

Safe to merge after addressing the run_git stderr omission; all other changes are docs/copy and well-guarded logic.

One P1 remains in run_git: failures from git config writes (the primary write path for the new feature) surface no diagnostic information, making debugging hard for users. All other findings are P2 or lower.

src/cli/install.rs — run_git stderr handling

Important Files Changed

Filename	Overview
src/cli/install.rs	Core logic for config-based hook install/uninstall; run_git uses .status() so stderr is discarded on failure, and warn_if_global_overlap is not called in the legacy branch — both are open threads flagged by prior review.
docs/getting_started.md	Getting started docs restructured to lead with hk install --global; adds manual .gitconfig example, per-repo alternative, and hkrc distinction. Content is accurate and well-organized.
docs/cli/install.md	CLI reference updated to promote --global as recommended path; description and flag help text updated consistently.
docs/cli/commands.json	Generated/serialized CLI help strings updated in sync with install.md and hk.usage.kdl; no issues.
hk.usage.kdl	KDL usage spec updated to match new doc strings; changes are consistent with other files.
docs/package.json	Added version and private:true fields — standard housekeeping to prevent accidental npm publish.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[hk install] --> B{--global?}
    B -- yes --> C{Git >= 2.54?}
    C -- no --> D[bail! requires Git 2.54+]
    C -- yes --> E[remove_config_entries --global]
    E --> F[write_config_hook --global for each DEFAULT_GLOBAL_EVENT]
    F --> G[Print success message]
    B -- no --> H[use_config_hooks = !legacy && git>=2.54]
    H --> I[Config::get — load hk.pkl]
    I --> J[remove_local_shims + remove_local_config_entries]
    J --> K{events empty?}
    K -- yes --> L[warn and return]
    K -- no --> M{use_config_hooks?}
    M -- yes --> N[install_local_config]
    N --> O[warn_if_global_overlap]
    M -- no --> P[install_local_shims]
    N --> Q[write_config_hook --local per event]
    P --> R[write shim file per event in .git/hooks/]

Comments Outside Diff (1)

src/cli/install.rs, line 276-281 (link)

run_git swallows stderr on failure

Command::status() does not capture stderr, so when a git config write fails (e.g. permission denied, locked config file) the error message only contains the subcommand args. remove_config_entries already uses Command::output() and surfaces stderr — the same pattern should be applied here so failures are actionable.

fn run_git(args: &[&str]) -> Result<()> {
    let output = Command::new("git").args(args).output()?;
    if !output.status.success() {
        bail!(
            "git {} failed (exit {}): {}",
            args.join(" "),
            output.status.code().unwrap_or(-1),
            String::from_utf8_lossy(&output.stderr).trim()
        );
    }
    Ok(())
}

_{Reviews (2): Last reviewed commit: "docs(install): recommend global hooks as..." | Re-trigger Greptile}

Lead with `hk install --global` in getting_started.md and the install CLI reference. The global install is a silent no-op in repos without an `hk.pkl`, so it's safe to enable everywhere and saves users from re-running `hk install` in every clone. Per-repo install is now framed as an alternative for pre-Git-2.54 or selective-repo use. Also renames the pre-existing "Global Configuration" section to "Global `hkrc` Configuration" to disambiguate from global hooks. Regenerates docs/cli from updated clap docstrings. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

@jdx

### 🐛 Bug Fixes - **(git)** skip untracked scan when HK_STASH_UNTRACKED=false by [@jdx](https://github.com/jdx) in [#861](#861) - **(run)** add post-commit and pre-rebase subcommands by [@jdx](https://github.com/jdx) in [#858](#858) ### 📚 Documentation - **(install)** recommend global hooks as primary setup path by [@jdx](https://github.com/jdx) in [#855](#855) - add cross-site announcement banner by [@jdx](https://github.com/jdx) in [#857](#857) - respect banner expires field by [@jdx](https://github.com/jdx) in [#862](#862) ### 🔍 Other Changes - vendor bats test helpers instead of git submodules by [@jdx](https://github.com/jdx) in [#859](#859) ### 📦️ Dependency Updates - bump communique to 1.0.3 by [@jdx](https://github.com/jdx) in [#863](#863) - update anthropics/claude-code-action digest to e58dfa5 by [@renovate[bot]](https://github.com/renovate[bot]) in [#864](#864)  --- > [!NOTE] > **Low Risk** > Low risk release bookkeeping only: version numbers, generated CLI/docs, and lockfile dependency bumps with no runtime logic changes. > > **Overview** > **Release prep for `v1.44.1`.** Updates `Cargo.toml`/`Cargo.lock` (including `libc`) and all version references in generated CLI docs (`docs/cli/*`, `hk.usage.kdl`). > > Refreshes documentation examples to reference the new `v1.44.1` Pkl package URLs and adds the `1.44.1` entry to `CHANGELOG.md`. > > <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit d7637d4. Bugbot is set up for automated code reviews on this repo. Configure [here](https://www.cursor.com/dashboard/bugbot).</sup>  Co-authored-by: mise-en-dev <123107610+mise-en-dev@users.noreply.github.com>

This MR contains the following updates: | Package | Update | Change | |---|---|---| | [hk](https://github.com/jdx/hk) | minor | `1.43.0` → `1.45.0` | MR created with the help of [el-capitano/tools/renovate-bot](https://gitlab.com/el-capitano/tools/renovate-bot). **Proposed changes to behavior should be submitted there as MRs.** --- ### Release Notes <details> <summary>jdx/hk (hk)</summary> ### [`v1.45.0`](https://github.com/jdx/hk/blob/HEAD/CHANGELOG.md#1450---2026-05-04) [Compare Source](jdx/hk@v1.44.3...v1.45.0) ##### 🚀 Features - **(builtins)** add `buildifier` format and lint built-ins by [@plx](https://github.com/plx) in [#896](jdx/hk#896) ##### 🐛 Bug Fixes - **(step)** only auto-batch when rendered command exceeds ARG\_MAX by [@jdx](https://github.com/jdx) in [#901](jdx/hk#901) ##### 📚 Documentation - thank Namespace for GitHub Actions runner support by [@jdx](https://github.com/jdx) in [#895](jdx/hk#895) ##### 🔍 Other Changes - **(ci)** use !cancelled() instead of always() for final job by [@jdx](https://github.com/jdx) in [#906](jdx/hk#906) - **(docs)** remove shrill.en.dev analytics script by [@jdx](https://github.com/jdx) in [#903](jdx/hk#903) - remove rust-cache from release jobs by [@jdx](https://github.com/jdx) in [#893](jdx/hk#893) - invert CLAUDE.md/AGENTS.md so AGENTS.md is canonical by [@jdx](https://github.com/jdx) in [#905](jdx/hk#905) - set dev profile debug to 1 by [@jdx](https://github.com/jdx) in [#907](jdx/hk#907) ##### 📦️ Dependency Updates - update anthropics/claude-code-action digest to [`fefa07e`](jdx/hk@fefa07e) by [@renovate\[bot\]](https://github.com/renovate\[bot]) in [#897](jdx/hk#897) - update jdx/mise-action digest to [`1648a78`](jdx/hk@1648a78) by [@renovate\[bot\]](https://github.com/renovate\[bot]) in [#898](jdx/hk#898) - update apple-actions/import-codesign-certs action to v7 by [@renovate\[bot\]](https://github.com/renovate\[bot]) in [#900](jdx/hk#900) - update autofix-ci/action action to v1.3.4 by [@renovate\[bot\]](https://github.com/renovate\[bot]) in [#899](jdx/hk#899) - lock file maintenance by [@renovate\[bot\]](https://github.com/renovate\[bot]) in [#908](jdx/hk#908) ##### New Contributors - [@plx](https://github.com/plx) made their first contribution in [#896](jdx/hk#896) ### [`v1.44.3`](https://github.com/jdx/hk/blob/HEAD/CHANGELOG.md#1443---2026-04-30) [Compare Source](jdx/hk@v1.44.2...v1.44.3) ##### 🐛 Bug Fixes - **(hook)** do not stage fixes when fail\_on\_fix=true by [@jdx](https://github.com/jdx) in [#892](jdx/hk#892) - use site domain for plausible data-domain by [@jdx](https://github.com/jdx) in [#886](jdx/hk#886) - make text-mode progress output usable in CI by [@jdx](https://github.com/jdx) in [#890](jdx/hk#890) ##### 📚 Documentation - prefix GitHub star count with ★ glyph by [@jdx](https://github.com/jdx) in [#883](jdx/hk#883) ##### 🔍 Other Changes - **(release)** dedupe sponsor section in release notes by [@jdx](https://github.com/jdx) in [#881](jdx/hk#881) - switch analytics from gtm/goatcounter to plausible by [@jdx](https://github.com/jdx) in [#885](jdx/hk#885) - migrate to namespace.so runners by [@jdx](https://github.com/jdx) in [#891](jdx/hk#891) ### [`v1.44.2`](https://github.com/jdx/hk/blob/HEAD/CHANGELOG.md#1442---2026-04-26) [Compare Source](jdx/hk@v1.44.1...v1.44.2) ##### 🐛 Bug Fixes - **(builtins)** silence pklr deprecation warnings on Builtins.pkl load by [@jdx](https://github.com/jdx) in [#880](jdx/hk#880) - **(ci)** serialize docs lint step by [@jdx](https://github.com/jdx) in [#874](jdx/hk#874) - **(config)** include main pkl path in cache fresh files by [@jdx](https://github.com/jdx) in [#879](jdx/hk#879) - **(docs)** stack banner message and link on mobile by [@jdx](https://github.com/jdx) in [#865](jdx/hk#865) - **(docs)** pin banner close button to top-right corner on mobile by [@jdx](https://github.com/jdx) in [#867](jdx/hk#867) ##### 📚 Documentation - **(site)** show release version and github stars by [@jdx](https://github.com/jdx) in [#872](jdx/hk#872) ##### 🔍 Other Changes - add pr-closer workflow by [@jdx](https://github.com/jdx) in [#876](jdx/hk#876) ##### 📦️ Dependency Updates - bump communique 1.0.3 → 1.0.4 by [@jdx](https://github.com/jdx) in [#868](jdx/hk#868) - update anthropics/claude-code-action digest to [`2da6cfa`](jdx/hk@2da6cfa) by [@renovate\[bot\]](https://github.com/renovate\[bot]) in [#869](jdx/hk#869) - update anthropics/claude-code-action digest to [`567fe95`](jdx/hk@567fe95) by [@renovate\[bot\]](https://github.com/renovate\[bot]) in [#870](jdx/hk#870) - bump communique to 1.1.2 by [@jdx](https://github.com/jdx) in [#875](jdx/hk#875) ### [`v1.44.1`](https://github.com/jdx/hk/blob/HEAD/CHANGELOG.md#1441---2026-04-24) [Compare Source](jdx/hk@v1.44.0...v1.44.1) ##### 🐛 Bug Fixes - **(git)** skip untracked scan when HK\_STASH\_UNTRACKED=false by [@jdx](https://github.com/jdx) in [#861](jdx/hk#861) - **(run)** add post-commit and pre-rebase subcommands by [@jdx](https://github.com/jdx) in [#858](jdx/hk#858) ##### 📚 Documentation - **(install)** recommend global hooks as primary setup path by [@jdx](https://github.com/jdx) in [#855](jdx/hk#855) - add cross-site announcement banner by [@jdx](https://github.com/jdx) in [#857](jdx/hk#857) - respect banner expires field by [@jdx](https://github.com/jdx) in [#862](jdx/hk#862) ##### 🔍 Other Changes - vendor bats test helpers instead of git submodules by [@jdx](https://github.com/jdx) in [#859](jdx/hk#859) ##### 📦️ Dependency Updates - bump communique to 1.0.3 by [@jdx](https://github.com/jdx) in [#863](jdx/hk#863) - update anthropics/claude-code-action digest to [`e58dfa5`](jdx/hk@e58dfa5) by [@renovate\[bot\]](https://github.com/renovate\[bot]) in [#864](jdx/hk#864) ### [`v1.44.0`](https://github.com/jdx/hk/blob/HEAD/CHANGELOG.md#1440---2026-04-23) [Compare Source](jdx/hk@v1.43.0...v1.44.0) ##### 🚀 Features - **(check)** implement --plan, --why, and --json by [@jdx](https://github.com/jdx) in [#848](jdx/hk#848) - **(cocogitto)** add cocogitto conventional commits config to hk builtin config by [@hituzi-no-sippo](https://github.com/hituzi-no-sippo) in [#838](jdx/hk#838) - **(git)** support GIT\_DIR/GIT\_WORK\_TREE for bare-repo dotfile managers by [@jdx](https://github.com/jdx) in [#847](jdx/hk#847) - **(install)** use Git 2.54 config-based hooks with --global support by [@jdx](https://github.com/jdx) in [#853](jdx/hk#853) ##### 🐛 Bug Fixes - use text progress in CI by [@jdx](https://github.com/jdx) in [#845](jdx/hk#845) ##### 📚 Documentation - generalize agent guidelines by [@jdx](https://github.com/jdx) in [#846](jdx/hk#846) - add releases nav and aube lock by [@jdx](https://github.com/jdx) in [#849](jdx/hk#849) ##### 🔍 Other Changes - **(release)** append en.dev sponsor blurb to release notes by [@jdx](https://github.com/jdx) in [#854](jdx/hk#854) - bump communique to 1.0.1 by [@jdx](https://github.com/jdx) in [#850](jdx/hk#850) ##### 📦️ Dependency Updates - update actions-rust-lang/setup-rust-toolchain digest to [`2b1f5e9`](jdx/hk@2b1f5e9) by [@renovate\[bot\]](https://github.com/renovate\[bot]) in [#832](jdx/hk#832) - update anthropics/claude-code-action digest to [`c3d45e8`](jdx/hk@c3d45e8) by [@renovate\[bot\]](https://github.com/renovate\[bot]) in [#833](jdx/hk#833) - update rust crate tokio to v1.52.1 by [@renovate\[bot\]](https://github.com/renovate\[bot]) in [#834](jdx/hk#834) - update actions/upload-pages-artifact action to v5 by [@renovate\[bot\]](https://github.com/renovate\[bot]) in [#835](jdx/hk#835) - update taiki-e/upload-rust-binary-action digest to [`f0d45ae`](jdx/hk@f0d45ae) by [@renovate\[bot\]](https://github.com/renovate\[bot]) in [#839](jdx/hk#839) - update rust crate clx to v2 by [@renovate\[bot\]](https://github.com/renovate\[bot]) in [#836](jdx/hk#836) - update anthropics/claude-code-action digest to [`0d2971c`](jdx/hk@0d2971c) by [@renovate\[bot\]](https://github.com/renovate\[bot]) in [#841](jdx/hk#841) - update anthropics/claude-code-action digest to [`38ec876`](jdx/hk@38ec876) by [@renovate\[bot\]](https://github.com/renovate\[bot]) in [#842](jdx/hk#842) - lock file maintenance by [@renovate\[bot\]](https://github.com/renovate\[bot]) in [#851](jdx/hk#851) </details> --- ### Configuration 📅 **Schedule**: (UTC) - Branch creation - At any time (no schedule defined) - Automerge - At any time (no schedule defined) 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever MR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this MR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this MR, check this box --- This MR has been generated by [Mend Renovate](https://github.com/renovatebot/renovate).

gemini-code-assist Bot reviewed Apr 23, 2026

View reviewed changes

greptile-apps Bot reviewed Apr 23, 2026

View reviewed changes

Comment thread src/cli/install.rs

Comment thread src/cli/install.rs

Comment thread src/cli/install.rs

jdx force-pushed the feat/config-based-hooks branch from 5305307 to c47b15a Compare April 23, 2026 17:46

jdx changed the title ~~feat(install): Git 2.54 config-based hooks with --global support~~ docs(install): recommend global hooks as primary setup path Apr 23, 2026

jdx enabled auto-merge (squash) April 23, 2026 17:56

jdx merged commit 1dce241 into main Apr 23, 2026
21 checks passed

jdx deleted the feat/config-based-hooks branch April 23, 2026 17:57

jdx mentioned this pull request Apr 23, 2026

chore: release v1.44.1 #856

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

docs(install): recommend global hooks as primary setup path#855

docs(install): recommend global hooks as primary setup path#855
jdx merged 1 commit intomainfrom
feat/config-based-hooks

jdx commented Apr 23, 2026 •

edited

Loading

Uh oh!

gemini-code-assist Bot left a comment

Uh oh!

gemini-code-assist Bot Apr 23, 2026

Uh oh!

gemini-code-assist Bot Apr 23, 2026

Uh oh!

gemini-code-assist Bot Apr 23, 2026

Uh oh!

gemini-code-assist Bot Apr 23, 2026

Uh oh!

greptile-apps Bot commented Apr 23, 2026 •

edited

Loading

Comments Outside Diff (1)

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

		@@ -1,33 +1,25 @@
		use crate::{Result, git_util};
		use crate::{Result, cli::install};

	use crate::{Result, cli::install};
	use crate::{Result, cli::install};
	use log::info;

Uh oh!

Conversation

jdx commented Apr 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

What changed

Why

Test plan

Uh oh!

gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

gemini-code-assist Bot Apr 23, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist Bot Apr 23, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist Bot Apr 23, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist Bot Apr 23, 2026

Choose a reason for hiding this comment

Uh oh!

greptile-apps Bot commented Apr 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Greptile Summary

Confidence Score: 4/5

Important Files Changed

Flowchart

Comments Outside Diff (1)

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

jdx commented Apr 23, 2026 •

edited

Loading

greptile-apps Bot commented Apr 23, 2026 •

edited

Loading