Skip to content

Commit 3188fe2

Browse files
committed
updates
1 parent 616885f commit 3188fe2

File tree

2 files changed

+12
-3
lines changed

2 files changed

+12
-3
lines changed

CLAUDE.md

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -84,6 +84,17 @@ Fast local feedback depends on using the same runner image and artifact flow as
8484

8585
Rebuild the runner image whenever dependencies change (e.g., adding packages to `docker/ci-runner/Dockerfile`), then rerun `act` to validate the workflows end-to-end.
8686

87+
### Required Local Validation Workflow
88+
89+
You **must** follow this exact sequence whenever you touch CI-related code or generated workflows:
90+
91+
1. Run the job's core command locally (e.g., `cargo fmt -- --check`, `cargo clippy --all-targets --all-features -- -D warnings`, or `cargo test --workspace`). Pick the job that will exercise your change.
92+
2. Run `task ci` and ensure it passes with no warnings.
93+
3. Re-run that same job via `act` (`timeout 600 act ... -j <job-name>`), using the local runner image so we mirror GitHub Actions.
94+
4. Only after the above succeed may you commit and push to GitHub.
95+
96+
Skipping or reordering these steps is not acceptable—they keep our feedback loop fast and avoid 10+ minute CI retries.
97+
8798
## Project Overview
8899

89100
`cigen` is a Rust CLI tool that generates CI pipeline configurations from templates. It integrates with Nx monorepo tooling and supports multiple CI providers starting with CircleCI.

plugins/provider-github/src/main.rs

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -787,9 +787,7 @@ fn install_protobuf_step() -> Mapping {
787787
);
788788
step.insert(
789789
Value::String("run".into()),
790-
Value::String(
791-
"set -e\nsudo apt-get update\nsudo apt-get install -y protobuf-compiler\n".into(),
792-
),
790+
Value::String("set -e\napt-get update\napt-get install -y protobuf-compiler\n".into()),
793791
);
794792
step
795793
}

0 commit comments

Comments
 (0)