Unified action for installing macos deps#7379

ShaharNaveh

Taken and modified from https://github.com/bevyengine/bevy/blob/162a7080911dc8b782475553fef8888c8d103655/.github/actions/install-linux-deps/action.yml

Summary by CodeRabbit

Chores

Refactored macOS build environment setup for improved consistency and maintainability across CI/CD workflows.

coderabbitai

📝 Walkthrough

Walkthrough

Introduces a new reusable GitHub Action to install macOS build dependencies and refactors the CI workflow to use this action instead of inline brew commands, consolidating dependency setup logic in two workflow jobs.

Changes

Cohort / File(s)	Summary
New macOS Dependency Action `.github/actions/install-macos-deps/action.yml`	Defines a composite GitHub Action with inputs for autoconf, automake, libtool, and openssl. Conditionally installs selected packages via brew based on input flags.
CI Workflow Updates `.github/workflows/ci.yaml`	Replaces inline brew install commands in `rust_tests` and `snippets_cpython` jobs with invocations of the new install-macos-deps action; adds `openssl: true` parameter in snippets_cpython job.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

Restructure CI with matrix approach and multi-feature support #7376: Both PRs modify macOS setup in .github/workflows/ci.yaml, with this PR introducing a reusable action to replace inline brew dependency installation commands.

Suggested reviewers

youknowone

Poem

🐰 A rabbit hops through workflows, neat,
Bundling brew commands—oh what a treat!
Dependencies dance on macOS so bright,
Reusable actions make CI just right! ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'Unified action for installing macos deps' accurately describes the main change: introducing a composite GitHub Action to standardize macOS dependency installation across workflows.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment

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 and usage tips.}

coderabbitai

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In @.github/actions/install-macos-deps/action.yml:
- Around line 39-47: The "Install macOS dependencies" step currently runs "brew
install" even when no formulae are selected, causing a failure; change the step
to compute the list of chosen inputs (inputs.autoconf, inputs.automake,
inputs.libtool, inputs.openssl) into a variable and skip running "brew install"
when that computed list is empty (i.e., no-op or echo a message), otherwise run
"brew install <packages>"; update the step's run script to perform this
conditional check so the action does nothing if all inputs are false.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: d572a819-438a-405a-b51f-df21ad724e9e

📥 Commits

Reviewing files that changed from the base of the PR and between abfb51e and 37b82ca.

📒 Files selected for processing (2)

.github/actions/install-macos-deps/action.yml
.github/workflows/ci.yaml

* Add `install-macos-deps` action * Use new action

ShaharNaveh added 2 commits March 7, 2026 12:43

Add install-macos-deps action

03f3d5e

Use new action

37b82ca

coderabbitai Bot reviewed Mar 7, 2026

View reviewed changes

youknowone approved these changes Mar 7, 2026

View reviewed changes

ShaharNaveh mentioned this pull request Mar 7, 2026

Unified action for installing Linux deps #7381

Merged

youknowone pushed a commit to youknowone/RustPython that referenced this pull request Mar 8, 2026

Unified action for installing macos deps (RustPython#7379) …

fa63408

* Add `install-macos-deps` action * Use new action

coderabbitai Bot mentioned this pull request Mar 9, 2026

Use matrix for cargo check #7388

Merged

youknowone pushed a commit to youknowone/RustPython that referenced this pull request Mar 22, 2026

Unified action for installing macos deps (RustPython#7379) …

a547e83

* Add `install-macos-deps` action * Use new action

coderabbitai Bot mentioned this pull request Mar 24, 2026

Cleanup release.yml a bit #7499

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Unified action for installing macos deps#7379