Skip to content

Conversation

@jfversluis
Copy link
Member

So that people with forks don't get unnecessary errors

Copilot AI review requested due to automatic review settings December 8, 2025 12:58
@jfversluis jfversluis added t/housekeeping ♻︎ area-infrastructure CI, Maestro / Coherency, upstream dependencies/versions labels Dec 8, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds repository owner checks to GitHub Actions workflows to prevent them from running unnecessarily in forked repositories, which would cause errors since forks typically lack the required secrets and permissions.

Key Changes:

  • Adds github.repository_owner == 'dotnet' condition to 8 workflow files
  • Uses proper logical operators (&&) with correct precedence to combine the new check with existing conditions
  • Ensures workflows only execute in the official dotnet organization repository

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.

Show a summary per file
File Description
.github/workflows/triage.yml Adds owner check with proper parentheses around existing OR conditions
.github/workflows/rebase.yml Adds owner check at the beginning of multi-line if condition
.github/workflows/maestro-changelog.yml Adds owner check alongside existing bot actor check
.github/workflows/inclusive-heat-sensor.yml Adds owner check as standalone job-level condition
.github/workflows/dotnet-format-daily.yml Adds owner check as standalone job-level condition
.github/workflows/dotnet-autoformat-pr-push.yml Adds owner check at the beginning of multi-line if condition
.github/workflows/backport.yml Adds owner check as standalone job-level condition
.github/workflows/add-remove-label-check-suites.yml Adds owner check to both add-label and remove-label jobs with proper parentheses

@jfversluis jfversluis merged commit 5662906 into main Dec 9, 2025
8 of 46 checks passed
@jfversluis jfversluis deleted the ghactions-dotnet-only branch December 9, 2025 10:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-infrastructure CI, Maestro / Coherency, upstream dependencies/versions t/housekeeping ♻︎

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants