-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Update list of collaborators #14086
Update list of collaborators #14086
Conversation
yashmayya
commented
Sep 26, 2024
•
edited
Loading
edited
- The collaborator list was initially added a few months ago in Add list of collaborators to asf.yaml #13346.
- The list needs to be periodically updated (every release or so) to take into account changes in contribution activity as well as existing collaborators becoming committers.
- The new list was generated with the same method used previously:
- Committers were manually filtered out from the above list (using the source of truth here - https://projects.apache.org/committee.html?pinot)
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #14086 +/- ##
============================================
+ Coverage 61.75% 63.99% +2.24%
- Complexity 207 1536 +1329
============================================
Files 2436 2596 +160
Lines 133233 143295 +10062
Branches 20636 21948 +1312
============================================
+ Hits 82274 91702 +9428
+ Misses 44911 44846 -65
- Partials 6048 6747 +699
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@@ -31,12 +31,12 @@ github: | |||
rebase: false | |||
collaborators: | |||
- sullis | |||
- shenyu0127 | |||
- tibrewalpratik17 | |||
- yashmayya |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess you used some kind of script to generate this list. Can you sort the list in alphabetical order? that would make it easier to actually understand who gets added and who gets removed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's sorted in order of # of commits in the past year (from the output of git shortlog --numbered --summary --since="1 year ago"
).