Skip to content

ci: drop dependency install from setup job and keep main runs - #2644

Merged
roottool merged 1 commit into
mainfrom
ci/lighten-setup-job
Oct 14, 2026
Merged

roottool merged 1 commit into
mainfrom
ci/lighten-setup-job

Conversation

@roottool

Copy link
Copy Markdown
Owner

Summary

Make the setup job lighter and stop cancelling in-progress CI runs on main.

Motivation

  • The setup job ran the full setup-environment action (Bun setup and cache restore) only to run dorny/paths-filter, which needs no dependencies.
  • cancel-in-progress: true also applied to main, so merging several PRs in a row cancelled the earlier runs.

Changes

  • Remove the Initialize step from the setup job. checkout stays, because on push the filter reads the git diff.
  • Set cancel-in-progress to ${{ github.ref != 'refs/heads/main' }}. Runs on pull request branches are still cancelled by newer pushes.

Testing

  • Format check (bun run format, lint:dprint, lint:oxfmt)
  • Setup completes without Initialize and Gatekeeper does not fail
  • After merge, consecutive pushes to main no longer cancel each other

Notes

Only ci.yml changes here, so it matches no path filter and Build and Lint are expected to be skipped on this PR. The concurrency change can only be observed on main after merging.

🤖 Generated with Claude Code

@roottool roottool self-assigned this Oct 14, 2026
@coderabbitai

coderabbitai Bot commented Oct 14, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 39f482d3-67b5-4d1e-a4a9-5924e77d9ded
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying portfolio with  Cloudflare Pages  Cloudflare Pages

Latest commit: 6a47c50
Status: ✅  Deploy successful!
Preview URL: https://bcd984bd.portfolio-6od.pages.dev
Branch Preview URL: https://ci-lighten-setup-job.portfolio-6od.pages.dev

View logs

@roottool
roottool merged commit 6dd1bb4 into main Oct 14, 2026
15 checks passed
@roottool
roottool deleted the ci/lighten-setup-job branch October 14, 2026 22:39
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