Skip to content
This repository was archived by the owner on Mar 14, 2023. It is now read-only.

Remove @kennytm from libs reviewers #419

Merged
merged 1 commit into from
Aug 8, 2022

Conversation

thomcc
Copy link
Member

@thomcc thomcc commented Aug 8, 2022

(I hope this isn't out of line at all -- while I don't really know them personally, I respect @kennytm a great deal, and have for a while now)

As noted in #416 (comment) @kennytm is pretty inactive. To quantify this, https://github.com/rust-lang/rust/pulls?q=is%3Apr+is%3Amerged+commenter%3Akennytm+%22approved+by+kennytm%22 says they've merged 3 reviews in the past year (EDIT: I think?). This is definitely an imperfect way to measure, but it seems infrequent enough that I think it's clear they're busy and the bot shouldn't auto-assign reviews to them.

(My motivation here is just that it ends up being a really bad contributor experience if they make a PR which gets assigned to an inactive reviewer -- this is most of why I go through the list of PR's assigned to them periodically)

@kennytm: Sorry if I've misinterpreted your availability! And to be clear, I think we'd be happy to re-add you to the review rotation if you'd like in the future. (And you can still r+ stuff, this PR just covers whether or not you're auto-assigned by the highfive bot)

CC @m-ou-se who probably should make the final call on this stuff, as the libs lead.

@Mark-Simulacrum Mark-Simulacrum merged commit 801e10a into rust-lang:master Aug 8, 2022
@thomcc thomcc deleted the drop-kennytm-libs branch August 12, 2022 05:18
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants