Skip to content

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

Open
jackluo923 wants to merge 1 commit into
ci-configurable-runnersfrom
ci-drop-docs-macos
Open

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

Conversation

@jackluo923

@jackluo923 jackluo923 commented Aug 1, 2026

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).

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

Auto reviews are disabled on base/target branches other than the default branch.

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: 90c0e84c-fbab-48e6-aedb-16bc7bfeec9a

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
jackluo923 marked this pull request as ready for review August 1, 2026 23:45
@jackluo923
jackluo923 requested a review from a team as a code owner August 1, 2026 23:45
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