Skip to content

Update require-codeowners.yml#14

Merged
jackdawm merged 1 commit intomainfrom
jackdawm-patch-2
May 1, 2025
Merged

Update require-codeowners.yml#14
jackdawm merged 1 commit intomainfrom
jackdawm-patch-2

Conversation

@jackdawm
Copy link
Copy Markdown
Contributor

@jackdawm jackdawm commented May 1, 2025

What was changed

Change the variable to pull the ref I actually meant to pull.

Why?

The github.ref_name was still resolving to main instead of the branch of the PR. Looking at the checkout action docs, this is the way I have to pull the correct reference. So it's still currently failing on PRs that are meant to fix this check, which is not desired behavior.

Checkout the correct branch this time.
@jackdawm jackdawm requested a review from a team as a code owner May 1, 2025 14:31
Copy link
Copy Markdown

@d3vinharrington d3vinharrington left a comment

Choose a reason for hiding this comment

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

LGTM

@jackdawm jackdawm merged commit de07568 into main May 1, 2025
6 checks passed
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