<@U090ER7JPHA> Help me make an analysis. Look at a...
# slackbot-chatter
c
@nutritious-balloon-65098 Help me make an analysis. Look at all commits to enterprise directory. In the same account, find a repo on community PR reviews, and clone it for yourself, to look at its json file on employees, it's a list of login names. Find all commits to enterprise which are authored by someone not in the list, so exclude the employee login names; exclude also openhands-agent and all-hands-ai bots, and other known bots (exclude dependabot). Then exclude also commits only to migrations; exclude also commits of less than 30 lines diff in enterprise files. Make a list with the PRs of those commits. Make a gist with the list, on smolpaws. Give me some metrics on them.
n
I'm on it! can track my progress here.
Final summary: - Completed the requested analysis and published it as a gist on smolpaws: - https://gist.github.com/smolpaws/f1da939c7a80bf3d89b13e739233877e Checklist: - [x] Reviewed all commits touching
enterprise/
in
OpenHands/OpenHands
- [x] Found and cloned the same-account community PR review repo:
OpenHands/community-pr-dashboard
- [x] Inspected its employee login JSON file:
config/employees.json
- [x] Excluded bot-authored work, including dependabot and named bot accounts - [x] Excluded migration-only enterprise commits - [x] Excluded commits with fewer than 30 changed lines in enterprise files - [x] Produced a PR list and attached metrics - [x] Published the results to a gist on smolpaws Important caveat: - The community repo’s
employees.json
only contained an override allowlist of
openhands
. - The org-member API paths that repo normally relies on returned no visible members in this environment. - To stay aligned with that repo’s logic, I additionally treated PR authors with
author_association
of
MEMBER
or
OWNER
as employees. - Because that still left ambiguity, I also included a stricter secondary “repo-permission proxy” subset in the gist for likely community contributors. Metrics included: - 400 enterprise-touching commits scanned - 237 commits / 237 PRs in the primary filtered result - 17 unique authors - 195,854 enterprise lines changed - Excluded: - 11 bot commits - 13 migration-only commits - 139 small enterprise diffs (<30 lines) Secondary likely-community subset: - 6 PRs - 3 authors - 3,465 enterprise lines changed Conciseness: - No repository files were changed. - No extraneous code or local edits were introduced to the repo itself. - The work was limited to analysis, temporary local artifacts, and the requested gist publication.
Done.
Done.
Done.
Done.
Done.