Attention is currently required from: Nicholas Chin.
Martin L Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/78184?usp=email )
Change subject: util/scripts: Add a script to find new users' commits on gerrit ......................................................................
Patch Set 2:
(3 comments)
Patchset:
PS1:
Needs an entry in `util/scripts/description. […]
Done
File util/scripts/find_new_user_commits.sh:
https://review.coreboot.org/c/coreboot/+/78184/comment/aa118cb8_0ce25a95 : PS1, Line 74: uploader.name + ", " + .currentPatchSet.uploader
Here we are using uploader, but the rest of the code implies we are using the owner field. […]
My effort here is really to make sure that people not experienced with coreboot and Gerrit are getting help. I figure the owner is really the important person here. If there's an experienced owner, an inexperienced author is already getting help.
https://review.coreboot.org/c/coreboot/+/78184/comment/ac022ac9_bad38215 : PS1, Line 100: commit_count="$(ssh -p 29418 "${GERRIT_USER}@${GERRIT_REPO}" gerrit query --format=JSON "limit:6 AND repo:coreboot AND owner:${owner_email} and status:merged" | : jq -Mr '.owner.name + ", " + .owner.email' | : grep -v "null|^," | : wc -l)"
I suppose the local git repo could be queried instead to get this information, though that ideally w […]
You're absolutely right - I just reused the query to gerrit since I was already using it to find the new open patches.