jameysharp opened PR #6084 from codeowners
to main
:
We want to make sure every contributor gets some kind of meaningful response in a timely fashion. To that end, this PR configures GitHub to auto-assign somebody to every newly-opened PR.
People must only be added to this file if they've agreed to this obligation. The details of what's expected are listed in the file. I'll only merge this if it's signed off by everyone listed in this initial version.
I've requested review from everyone; please either approve or request to be removed. Or suggest changes, of course; I'd particularly like to know if my description of the expectations aren't clear or don't seem right, or if we should slice up the repository differently.
I considered using the bytecodealliance/cranelift-core and wasmtime-core teams for this, but they have too many people and I wanted to make sure I was getting opt-in from everyone. Eventually I think we should create a few new teams specifically for this purpose, but that requires bytecodealliance org admin rights as far as I can tell, and I wanted to get the general shape of this sorted out before dealing with that.
I'd like to ask some more people to join some of these lists, but for example @itsrainy is out this week so I couldn't check with them.
jameysharp requested elliottt for a review on PR #6084.
jameysharp requested alexcrichton for a review on PR #6084.
jameysharp requested pchickey for a review on PR #6084.
jameysharp requested cfallin for a review on PR #6084.
jameysharp requested fitzgen for a review on PR #6084.
pchickey submitted PR review.
pchickey created PR review comment:
I think its worthwhile to break down the /crates further:
I will volunteer for component-macro, component-util, environ, fiber, runtime, test-programs, wasi, wasi-common, wasmtime, wiggle, wit-bindgenwasi-nn and wasi-threads are maintained by @abrown, I will be a secondary on these in case he is unavailable
winch is @saulecabrera
cfallin submitted PR review.
alexcrichton submitted PR review.
elliottt submitted PR review.
fitzgen submitted PR review.
jameysharp submitted PR review.
jameysharp created PR review comment:
I wanted to limit the number of people in this initial PR so we can start evaluating how well this works for the project as a whole. If Andrew or Saúl would like to make the time commitment described in this file, we can certainly add them later.
Also, keep in mind that I'm not expecting the people in this file to necessarily be the ideal reviewers for the areas covered. Instead they should be people who know who the right reviewer is and, more importantly, can quickly get a response out so a contributor isn't waiting a long time. So it isn't as important to be precise about who's handling which area. I think it's more important to keep the lists simple so it's easy to tell at a glance who might get handed what.
In particular, given that you've volunteered for a third of the directories in /crates, I think you have a good idea who to hand reviews off to for the rest. So if it's alright with you, I'll just add you to the rotation for the whole Wasmtime group and drop the WASI category.
saulecabrera submitted PR review.
saulecabrera created PR review comment:
@jameysharp I'm good with the time commitment described in this document and to be added to this list whenever you feel it's convenient. Thanks!
jameysharp updated PR #6084 from codeowners
to main
.
jameysharp merged PR #6084.
Last updated: Dec 23 2024 at 12:05 UTC