Skip to content

ci: only run Windows/Mac CI on main or with ci: full label#1062

Closed
Boshen wants to merge 1 commit intomainfrom
ci/skip-expensive-runners
Closed

ci: only run Windows/Mac CI on main or with ci: full label#1062
Boshen wants to merge 1 commit intomainfrom
ci/skip-expensive-runners

Conversation

@Boshen
Copy link
Member

@Boshen Boshen commented Mar 20, 2026

Summary

  • Skip Windows and macOS CI matrix entries on PRs unless the ci: full label is present
  • Linux CI continues to run on all PRs unconditionally
  • All OS matrix entries still run on main push and workflow_dispatch

🤖 Generated with Claude Code

@netlify
Copy link

netlify bot commented Mar 20, 2026

Deploy Preview for viteplus-preview canceled.

Name Link
🔨 Latest commit 5d3c57e
🔍 Latest deploy log https://app.netlify.com/projects/viteplus-preview/deploys/69bcd34c557aa400084ddcd9

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1a6e757fd2

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: e387052aa1

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

- Remove `detect-changes` job (paths-filter was unreliable)
- Jobs now gate on `optimize-ci` directly
- Skip Windows and macOS matrix entries on PRs unless `ci: full` label is present
- Linux CI continues to run on all PRs unconditionally

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@Boshen Boshen force-pushed the ci/skip-expensive-runners branch from 6117e08 to 5d3c57e Compare March 20, 2026 04:55
@Boshen Boshen marked this pull request as draft March 20, 2026 07:33
@Boshen Boshen closed this Mar 20, 2026
@Boshen Boshen deleted the ci/skip-expensive-runners branch March 20, 2026 10:07
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