Skip to content

ci: Run clippy and the Rust tests in one job - #806

Merged
Ziinc merged 3 commits into
claude/speed-up-ci-aoedg6-preview-screenshotsfrom
claude/speed-up-ci-aoedg6-rust-job
Oct 12, 2026
Merged

Ziinc merged 3 commits into
claude/speed-up-ci-aoedg6-preview-screenshotsfrom
claude/speed-up-ci-aoedg6-rust-job

Conversation

@Ziinc

@Ziinc Ziinc commented Oct 12, 2026 •

Copy link
Copy Markdown
Collaborator

Requested by Zn · project thread

  • Merges verify-rust / clippy into test-rust: clippy runs first, then nextest runs even if clippy fails. One runner and one setup instead of two.
  • warm-caches.yml builds the clippy and test artifacts into the one test cache that the merged job restores.
  • Branch protection: replace verify-rust / clippy and test-rust / test / ubuntu-22.04 with test-rust / clippy + test / ubuntu-22.04.
    🤖 Generated with Claude Code
    https://claude.ai/code/session_01Uwdgq12RrXjifDLXargwye
    Generated by Claude Code

Each takes 1-2 min on a warm cache, so the separate clippy job only paid
for a second runner and its setup. Clippy runs first; the tests run even
if it fails. warm-caches.yml builds both into the shared 'test' cache.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Uwdgq12RrXjifDLXargwye
@Ziinc Ziinc self-assigned this Oct 12, 2026
main fails check:format on the addPromptHistory call.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Uwdgq12RrXjifDLXargwye

Ziinc commented Oct 12, 2026 •

Copy link
Copy Markdown
Collaborator Author

verify / ubuntu-22.04 failed on check:format in src/components/Dashboard.tsx, which this PR doesn't touch: main itself is unformatted there since #669. Fix is #811; I ported the identical change into this branch (it no-ops once #811 merges) and restacked #807 and #808 on top.


Generated by Claude Code

A change to the Rust job's definition never ran it, so it couldn't verify
itself. The js filter already lists ci.yml.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Uwdgq12RrXjifDLXargwye

Ziinc commented Oct 12, 2026

Copy link
Copy Markdown
Collaborator Author

test-js / test:integration failed on 2 tests in test/integration/workspace/create-pr.test.tsx (pushWorkspaceToRemote not called; not_on_remote still true). This PR changes only workflow files, and #807 and #808, which contain this exact diff plus more, pass the same job. That points to timing flakiness in that test, not this PR. Re-running the failed job once.


Generated by Claude Code

@Ziinc
Ziinc merged commit 4732088 into claude/speed-up-ci-aoedg6-preview-screenshots Oct 12, 2026
10 of 11 checks passed
@Ziinc
Ziinc deleted the claude/speed-up-ci-aoedg6-rust-job branch October 12, 2026 06:07
Ziinc added a commit that referenced this pull request Oct 12, 2026
Co-authored-by: Claude <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants