Skip to content

ci(perf): cockpit-examples-build builds only affected caps on PR#554

Merged
blove merged 1 commit into
mainfrom
claude/build-affected
May 28, 2026
Merged

ci(perf): cockpit-examples-build builds only affected caps on PR#554
blove merged 1 commit into
mainfrom
claude/build-affected

Conversation

@blove
Copy link
Copy Markdown
Contributor

@blove blove commented May 28, 2026

Summary

cockpit-examples-build ran nx run-many --projects='cockpit-*-angular' on every PR — building all 32 cockpit angular projects (~5m28s) regardless of what changed.

Now:

  • PR: builds only the cockpit-*-angular projects nx considers affected (base..head). 1-cap PR → 1 build; libs/chat fanout → all 32; docs-only → fast no-op.
  • push to main: unchanged full build of all 32 (pre-deploy safety net).

This is a production nx build, distinct from cockpit-e2e's nx serve (dev-server) — not redundant, so the job stays; it just scopes its work on PRs. Adds fetch-depth: 0 to checkout (required for nx affected base..head).

Test plan

  • YAML lint clean
  • This PR (touches only ci.yml → 0 affected cockpit caps) → build step is a fast no-op
  • A future cap-touching PR → builds only that cap

🤖 Generated with Claude Code

`cockpit-examples-build` ran `nx run-many --projects='cockpit-*-angular'`
on every PR — building all 32 cockpit angular projects (~5m28s)
regardless of what changed.

Now:
- PR: build only the cockpit-*-angular projects nx considers affected
  (base..head). A 1-cap PR builds 1 cap; a libs/chat fanout builds all.
  Empty affected set → fast no-op.
- push to main: unchanged full build of all 32 (pre-deploy safety net).

This is a production `nx build`, distinct from cockpit-e2e's `nx serve`
(dev-server) — not redundant, so the job stays; it just scopes its work.

Adds fetch-depth: 0 to checkout (required for nx affected base..head).
@vercel
Copy link
Copy Markdown

vercel Bot commented May 28, 2026

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

Project Deployment Actions Updated (UTC)
threadplane Ready Ready Preview, Comment May 28, 2026 4:22am

Request Review

@blove blove merged commit c01435b into main May 28, 2026
48 checks passed
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.

1 participant