Skip to content

Conversation

@timotheeguerin
Copy link
Member

From the doc this seems to be the way to define catch all. Currently we are getting added to all csharp emitter PRs

https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-code-owners

Particularly this part


# These owners will be the default owners for everything in
# the repo. Unless a later match takes precedence,
# @global-owner1 and @global-owner2 will be requested for
# review when someone opens a pull request.
*       @global-owner1 @global-owner2

@azure-sdk
Copy link
Collaborator

No changes needing a change description found.

@azure-sdk
Copy link
Collaborator

You can try these changes at https://cadlplayground.z22.web.core.windows.net/prs/3505/

Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/prs/3505/

@timotheeguerin timotheeguerin enabled auto-merge June 3, 2024 16:23
@timotheeguerin timotheeguerin added this pull request to the merge queue Jun 3, 2024
Merged via the queue into main with commit 12344d1 Jun 3, 2024
@timotheeguerin timotheeguerin deleted the timotheeguerin-patch-2 branch June 3, 2024 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants