Skip to content

Add some codeowners#4615

Merged
radical merged 3 commits intomicrosoft:mainfrom
radical:codeowners
Jun 20, 2024
Merged

Add some codeowners#4615
radical merged 3 commits intomicrosoft:mainfrom
radical:codeowners

Conversation

@radical
Copy link
Copy Markdown
Member

@radical radical commented Jun 20, 2024

Microsoft Reviewers: Open in CodeFlow

@ghost ghost added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Jun 20, 2024
@radical radical requested a review from danmoseley June 20, 2024 20:35
@radical
Copy link
Copy Markdown
Member Author

radical commented Jun 20, 2024

I added a few, but please free to add more.

Copy link
Copy Markdown
Member

@eerhardt eerhardt left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread CODEOWNERS
@@ -1 +1,16 @@
/eng/SourceBuild* @dotnet/source-build-internal No newline at end of file
/eng/SourceBuild* @dotnet/source-build-internal
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Why is github complaining about this? This line is same as in the other repos - https://github.com/dotnet/sdk/blob/7822425c3ea35e0bb46e57fad11663e7cbf9f850/CODEOWNERS#L106

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Unknown owner on line 1: make sure the team @dotnet/source-build-internal exists, is publicly visible, and has write access to the repository
/eng/SourceBuild* @dotnet/source-build-internal

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Has that team been given any rights to this repo? I don't see it in the list.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I guess not! Who can fix this?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I added that team with write permission

@radical radical enabled auto-merge (squash) June 20, 2024 23:03
@radical radical merged commit 80a54dc into microsoft:main Jun 20, 2024
@radical radical deleted the codeowners branch June 20, 2024 23:14
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 21, 2024
@github-actions github-actions Bot added area-codeflow for labeling automated codeflow. intentionally a different color! and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Mar 10, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-codeflow for labeling automated codeflow. intentionally a different color!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants