Skip to content

Update require-codeowners.yml#12

Merged
jackdawm merged 1 commit intomainfrom
jackdawm-patch-1
Apr 28, 2025
Merged

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

Conversation

@jackdawm
Copy link
Copy Markdown
Contributor

What was changed

Actually make this return non-0 if a file named CODEOWNERS does not exist.

Why?

My "clever" way did not actually return non-0 when the sparse checkout has no files.

Checklist

  1. Closes

  2. How was this tested:

  1. Any docs updates needed?

@jackdawm jackdawm requested a review from a team as a code owner April 28, 2025 14:10
@jackdawm jackdawm merged commit 596869c into main Apr 28, 2025
6 checks passed
@jackdawm jackdawm deleted the jackdawm-patch-1 branch April 28, 2025 20:12
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