Skip to content

*: exclude make bazel_lint_changed from default AGENTS local flow #66678

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
ti-chi-bot merged 3 commits into from
Mar 5, 2026
Merged

*: exclude make bazel_lint_changed from default AGENTS local flow #66678

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
78dde66
remove the "MUST run `make bazel_lint_changed`" from agents.md, it's …
D3Hunter Mar 4, 2026
7453bc7
Update AGENTS.md
D3Hunter Mar 4, 2026
ee110ad
change
D3Hunter Mar 4, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions AGENTS.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ This file provides guidance to agents working in this repository.
| RealTiKV tests | MUST start playground in background, run tests, then clean up playground/data (see `docs/agents/testing-flow.md`). |
| Bug fix | MUST add a regression test and verify it fails before fix and passes after fix. |
| Fmt-only PR | MUST NOT run costly `realtikvtest`; local compilation is enough. |
| Before finishing | MUST run `make bazel_lint_changed` if there are code changes. SHOULD self-review diff quality before finishing. |
Before finishing | MUST run `make lint` if there are code changes. SHOULD self-review diff quality before finishing. |

### Skills

Expand Down Expand Up @@ -87,9 +87,10 @@ make bazel_bin
make gogenerate # optional: regenerate generated code
go mod tidy # optional: if go.mod/go.sum changed
git fetch origin --prune
make bazel_lint_changed # Optional: skip this step if the resolved Bazel target is //:all.
```

`make bazel_lint_changed` is intentionally excluded from the default local flow because it can be slow and resource-intensive on local macOS environments. Agents MUST NOT run `make bazel_lint_changed` unless the user explicitly requests it.

## Task -> Validation Matrix

Use the smallest set that still proves correctness.
Expand Down
⚡