mbox series

[net,0/9] MAINTAINERS: remove inactive folks from networking

Message ID 20210111052759.2144758-1-kuba@kernel.org (mailing list archive)
Headers show
Series MAINTAINERS: remove inactive folks from networking | expand

Message

Jakub Kicinski Jan. 11, 2021, 5:27 a.m. UTC
Hi!

This series intends to remove some most evidently inactive maintainers.

To make maintainers' lives easier we're trying to nudge people
towards CCing all the relevant folks on patches, in an attempt
to improve review rate. We have a check in patchwork which validates
the CC list against get_maintainers.pl. It's a little awkward, however,
to force people to CC maintainers who we haven't seen on the mailing
list for years. This series removes from maintainers folks who didn't
provide any tag (incl. authoring a patch) in the last 5 years.
To ensure reasonable signal to noise ratio we only considered
MAINTAINERS entries which had more than 100 patches fall under
them in that time period.

All this is purely a process-greasing exercise, I hope nobody
sees this series as an affront. Most folks are moved to CREDITS,
a couple entries are simply removed. 

The following inactive maintainers are kept, because they indicated
the intention to come back in the near future:

 - Veaceslav Falico (bonding)
 - Christian Benvenuti (Cisco drivers)

Patches in this series contain report from a script which did
the analysis. Big thanks to Jonathan Corbet for help and writing
the script (although I feel like I used it differently than Jon
may have intended ;)). The output format is thus:

 Subsystem $name
  Changes $reviewed / $total ($percent%)           // how many changes to the subsystem had at least one ack/review
  Last activity: $date_of_most_recent_patch
  $maintainer/reviewer1:
    Author $last_commit_authored_by_the_person $how_many_in_5yrs
    Committer $last_committed $how_many
    Tags $last_tag_like_review_signoff_etc $how_many
  $maintainer/reviewer2:
    Author $last_commit_authored_by_the_person $how_many_in_5yrs
    Committer $last_committed $how_many
    Tags $last_tag_like_review_signoff_etc $how_many
  Top reviewers: // Top 3 reviewers (who are not listed in MAINTAINERS)
    [$count_of_reviews_and_acks]: $email
  INACTIVE MAINTAINER $name   // maintainer / reviewer who has done nothing in last 5yrs

Jakub Kicinski (9):
  MAINTAINERS: altx: move Jay Cliburn to CREDITS
  MAINTAINERS: net: move Alexey Kuznetsov to CREDITS
  MAINTAINERS: vrf: move Shrijeet to CREDITS
  MAINTAINERS: ena: remove Zorik Machulsky from reviewers
  MAINTAINERS: tls: move Aviad to CREDITS
  MAINTAINERS: mtk-eth: remove Felix
  MAINTAINERS: ipvs: move Wensong Zhang to CREDITS
  MAINTAINERS: dccp: move Gerrit Renker to CREDITS
  MAINTAINERS: skge/sky2: move Mirko Lindner to CREDITS

 CREDITS     | 28 ++++++++++++++++++++++++++++
 MAINTAINERS | 11 +----------
 2 files changed, 29 insertions(+), 10 deletions(-)