Skip to content

Conversation

@krlvi
Copy link
Member

@krlvi krlvi commented Jan 24, 2026

No description provided.

Copilot AI review requested due to automatic review settings January 24, 2026 17:18
@vercel
Copy link

vercel bot commented Jan 24, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Review Updated (UTC)
gitbutler-web Ignored Ignored Preview Jan 24, 2026 5:28pm

Request Review

@krlvi krlvi enabled auto-merge January 24, 2026 17:18
@github-actions github-actions bot added the rust Pull requests that update Rust code label Jan 24, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the clippy command documentation in crates/but/agents.md to include the --all-targets flag, ensuring consistency with the clippy commands documented elsewhere in the repository (.github/copilot-instructions.md and custom-instructions/repo/.github/copilot-instructions.md).

Changes:

  • Added --all-targets flag to the clippy auto-fix command in the but CLI agent documentation


* use `cargo fmt --check --all` to check for formatting issues.
* use `cargo clippy --fix --allow-dirty` to auto-fix clippy errors.
* use `cargo clippy --all-targets --fix --allow-dirty` to auto-fix clippy errors.
Copy link

Copilot AI Jan 24, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is an extra space between --fix and --allow-dirty. It should be --fix --allow-dirty (single space) to match the formatting convention used elsewhere in the codebase.

Suggested change
* use `cargo clippy --all-targets --fix --allow-dirty` to auto-fix clippy errors.
* use `cargo clippy --all-targets --fix --allow-dirty` to auto-fix clippy errors.

Copilot uses AI. Check for mistakes.
@krlvi krlvi merged commit 7ba8c97 into master Jan 24, 2026
23 checks passed
@krlvi krlvi deleted the kv-branch-87 branch January 24, 2026 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rust Pull requests that update Rust code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants