Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

updating area ownership within mono and coreclr #97571

Merged
merged 2 commits into from
Jan 29, 2024

Conversation

jeffschwMSFT
Copy link
Member

per our discussion, here are the updated area ownership for mono and coreclr

the sub area owners likely need feedback/updates

cc @agocke, @lewing, @steveisok, @vitek-karas, @tommcdon, @mangod9

@ghost
Copy link

ghost commented Jan 26, 2024

Tagging subscribers to this area: @dotnet/area-meta
See info in area-owners.md if you want to be subscribed.

Issue Details

per our discussion, here are the updated area ownership for mono and coreclr

the sub area owners likely need feedback/updates

cc @agocke, @lewing, @steveisok, @vitek-karas, @tommcdon, @mangod9

Author: jeffschwMSFT
Assignees: -
Labels:

area-Meta

Milestone: -

Copy link
Member

@lambdageek lambdageek left a comment

Choose a reason for hiding this comment

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

looks ok; question about EnC in mono, and a suggestion about the mono GC

Comment on lines +26 to +27
| area-CrossGen/NGEN-coreclr | @steveisok | @dotnet/crossgen-contrib | |
| area-crossgen2-coreclr | @steveisok | @dotnet/crossgen-contrib | |
Copy link
Member

Choose a reason for hiding this comment

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

@steveisok do we need to add Fan, Zoltan and Kate to https://github.com/orgs/dotnet/teams/crossgen-contrib ?

docs/area-owners.md Outdated Show resolved Hide resolved
docs/area-owners.md Outdated Show resolved Hide resolved
| area-Host | @agocke | @jeffschwMSFT @vitek-karas @vsadov | Issues with dotnet.exe including bootstrapping, framework detection, hostfxr.dll and hostpolicy.dll |
| area-HostModel | @agocke | @vitek-karas | |
| area-ILTools-coreclr | @JulieLeeMSFT | @BruceForstall @dotnet/jit-contrib | |
| area-Infrastructure | @agocke | @jeffschwMSFT @MichaelSimons | |
| area-Infrastructure-coreclr | @agocke | @jeffschwMSFT @trylek | |
| area-Infrastructure-coreclr | @agocke | @jeffschwMSFT | |
| area-Infrastructure-installer | @MichaelSimons | @NikolaMilosavljevic | |
| area-Infrastructure-libraries | @ericstj | @dotnet/area-infrastructure-libraries | Covers:<ul><li>Packaging</li><li>Build and test infra for libraries in dotnet/runtime repo</li><li>VS integration</li></ul><br/> |
| area-Infrastructure-mono | @steveisok | @directhex | |
Copy link
Member

Choose a reason for hiding this comment

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

@jeffschwMSFT @lewing @steveisok who should this change to?

Copy link
Member Author

Choose a reason for hiding this comment

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

I left Jo in that space as he has a lot of knowledge and still works in the repo. I can remove him for now. We discussed a need for some infrastructure triage to sort this out (which is pending)

Copy link
Member

Choose a reason for hiding this comment

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

We can keep Jo listed as a consultant but not as owner / responsible party.

@jeffschwMSFT
Copy link
Member Author

I am going to merge for now, but we can make additional changes as necessary (eg. Jo)

@jeffschwMSFT jeffschwMSFT merged commit 902d514 into dotnet:main Jan 29, 2024
12 checks passed
@jeffschwMSFT jeffschwMSFT deleted the update_owners branch January 29, 2024 16:55
@github-actions github-actions bot locked and limited conversation to collaborators Feb 29, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants