Skip to content

Let Callers Choose the Runner for the Coverage Workflow - #33

Merged
PSchmiedmayer merged 1 commit into
mainfrom
feature/coverage-runner-label
Aug 17, 2026
Merged

Let Callers Choose the Runner for the Coverage Workflow#33
PSchmiedmayer merged 1 commit into
mainfrom
feature/coverage-runner-label

Conversation

@PSchmiedmayer

Copy link
Copy Markdown
Contributor

♻️ Current situation & Problem

coverage.yml pins macos-26. Merging xcresult bundles does need macOS, but the runner is not something the caller can choose, so a repository with its own macOS runners still ends up on a GitHub-hosted one. On a private repository that is billed time, and the job fails before it starts when none is available.

This is the last shared workflow without a runner input.

⚙️ Release Notes

  • Accept runs_on_labels in coverage.yml, defaulting to macos-26 so nothing changes for callers that pass nothing.

📚 Documentation

The input description notes that merging xcresult bundles needs macOS, so a caller does not point it at a Linux runner by accident.

✅ Testing

  • actionlint
  • The workflow parses and exposes the new input with the previous runner as the default

Code of Conduct & Contributing Guidelines

By creating and submitting this pull request, you agree to follow our Code of Conduct and Contributing Guidelines:

Copilot AI lite review requested due to automatic review settings August 17, 2026 07:11

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@coderabbitai

coderabbitai Bot commented Aug 17, 2026

Copy link
Copy Markdown

Important

Review available on request

  • 🔍 Trigger review

Reviews should be triggered manually for repositories with fewer than 10 stars. Select Trigger review above or comment @coderabbitai review to review the latest changes. For a full review, comment @coderabbitai full review.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 36742c31-f7d8-467d-b889-2cccc4fbd7b7


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.

@PSchmiedmayer
PSchmiedmayer merged commit 86a59a7 into main Aug 17, 2026
6 checks passed
@PSchmiedmayer
PSchmiedmayer deleted the feature/coverage-runner-label branch August 17, 2026 07:18
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