Skip to content

ci: Drop the standalone macOS docs build. - #2446

Closed
jackluo923 wants to merge 1 commit into
y-scope:ci-configurable-runnersfrom
jackluo923:ci/drop-docs-macos
Closed

ci: Drop the standalone macOS docs build.#2446
jackluo923 wants to merge 1 commit into
y-scope:ci-configurable-runnersfrom
jackluo923:ci/drop-docs-macos

Conversation

@jackluo923

Copy link
Copy Markdown
Member

Description

Removes .github/workflows/clp-docs-macos.yaml. task docs:site is OS-agnostic (Sphinx + a broken-links check in a uv venv), so the macOS leg duplicated the Linux clp-docs.yaml run without adding OS-specific signal, and its output was never published (the upload step was gated on ubuntu-24.04). The Linux-only clp-docs.yaml already builds and publishes docs; the other macOS legs remain (clp-uv-checks-macos, clp-lint-macos, clp-s-generated-code-checks-macos, clp-core-build-macos).

Stacked on #2445.

Checklist

Validation performed

  • yamllint clean; actionlint reports no new findings (only the pre-existing SC2046 chown warnings repo-wide).
  • All CI checks on the branch pass (to be confirmed once the run completes).

`task docs:site` is OS-agnostic (Sphinx + a broken-links check in a uv venv), so the macOS leg duplicated the Linux `clp-docs.yaml` run without adding OS-specific signal, and it was never published (the upload step was gated on `ubuntu-24.04`). The Linux-only `clp-docs.yaml` already covers docs building and publishing; removing `clp-docs-macos.yaml` drops one redundant macOS job. macOS coverage for the other gated legs is unaffected (clp-uv-checks-macos, clp-lint-macos, clp-s-generated-code-checks-macos, clp-core-build-macos all remain).
@coderabbitai

coderabbitai Bot commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 2a722d78-7086-4a18-833f-043f91801f5e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

@jackluo923

Copy link
Copy Markdown
Member Author

Closing in favor of a new PR with the head branch on this repo (ci-drop-docs-macos) — a fork head can't be retargeted to an upstream base, and stacking works cleanly only with both branches on the same repo. Reopening shortly.

@jackluo923 jackluo923 closed this Aug 1, 2026
@jackluo923
jackluo923 deleted the ci/drop-docs-macos branch August 1, 2026 22:58
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