address auto-review

Co-authored-by: Sebastian Thiel <sebastian.thiel@icloud.com>

Author: codex

.github/workflows/push.yaml

@@ -304,7 +304,6 @@ jobs:
       # It's intentional to use 'name equals run-script' so it's easy to re-run locally on failure.
       - run: cargo test -p but
       - run: make test-modern-but
-      - run: cargo test -p but
       - run: cargo test -p but-server
       - name: test vendored
         run: |

crates/but/src/command/commit/file.rs

@@ -200,8 +200,8 @@ fn find_stack_id_for_branch_with_perm(
 
 /// Refresh the workspace commit when legacy workspace state is available.
 /// TODO: remove this as it shouldn't be needed - all the functions it calls use the rebase engine which takes care of that.
-fn legacy_update_workspace_commit(ctx: &mut Context) -> Result<()> {
+fn legacy_update_workspace_commit(_ctx: &mut Context) -> Result<()> {
     #[cfg(feature = "legacy")]
-    gitbutler_branch_actions::update_workspace_commit(ctx, false)?;
+    gitbutler_branch_actions::update_workspace_commit(_ctx, false)?;
     Ok(())
 }