Another thing we could try and do (if folks would be down to try) is it have not actually ping, but suggest the potential usernames to ping to the user (e.g. say suggested reviewers you _may wish to ping_ and then list)?
On Mon, Jul 30, 2018 at 10:45 PM, Holden Karau <hol...@pigscanfly.ca> wrote: > > On Mon, Jul 30, 2018 at 10:22 PM, Reynold Xin <r...@databricks.com> wrote: > >> I like the idea of this bot, but I'm somewhat annoyed by it. I have >> touched a lot of files and wrote a lot of the original code. Everyday I >> wake up I get a lot of emails from this bot. >> > We could blacklist the existing PMC (or add a rate limit)? > >> >> Also if we are going to use this, can we rename the bot to something like >> spark-bot, rather than holden's personal bot? >> > I originally did that, but GitHub told me I could only have one personal > and one bot account. If someone else registered the spark-mention-bot I'd > be happy to switch it to that. > >> >> On Mon, Jul 30, 2018 at 10:18 PM Hyukjin Kwon <gurwls...@gmail.com> >> wrote: >> >>> > That being said the folks being pinged are not just committers. >>> >>> I doubt it because only pinged ones I see are all committers and that's >>> why I assumed the pinging is based on who committed the PR (which implies >>> committer only). >>> Do you maybe have some examples where non-committers were pinged? Looks >>> at least, (almost?) all of them are committers and something needs to be >>> fixed even if so. >>> >>> I recently argued about pinging things before - sounds it matters if it >>> annoys. Since pinging is completely optional and cc'ing someone else might >>> need other contexts not >>> only assuming from the blame and who committed this, I am actually not >>> super happy with that pinging for now. I was slightly supportive for this >>> idea but now I actually slightly >>> became negative on this after observing how it goes in practice. >>> >>> I wonder how other people think on this. >>> >>> >>> >>> 2018년 7월 31일 (화) 오후 12:33, Holden Karau <hol...@pigscanfly.ca>님이 작성: >>> >>>> So CODEOWNERS is limited to committers by GitHub. We can definitely >>>> modify the config file though and I'm happy to write some custom logic if >>>> it helps support our needs. We can also just turn it off if it's too noisey >>>> for folks in general. >>>> >>>> That being said the folks being pinged are not just committers. The >>>> hope is to get more code authors who aren't committers involved in the >>>> reviews and then eventually become committers. >>>> >>>> On Mon, Jul 30, 2018, 9:09 PM Hyukjin Kwon <gurwls...@gmail.com> wrote: >>>> >>>>> *reviewers: I mean people who committed the PR given my observation. >>>>> >>>>> 2018년 7월 31일 (화) 오전 11:50, Hyukjin Kwon <gurwls...@gmail.com>님이 작성: >>>>> >>>>>> I was wondering if we can leave the configuration open and accept >>>>>> some custom configurations, IMHO, because I saw some people less related >>>>>> or >>>>>> less active are consistently pinged. Just started to get worried if they >>>>>> get annoyed by this. >>>>>> Also, some people could be interested in few specific areas. They >>>>>> should get pinged too. >>>>>> Also, assuming from people pinged, seems they are reviewers (which >>>>>> basically means committers I guess). Was wondering if there's a big >>>>>> difference between codeowners and bots. >>>>>> >>>>>> >>>>>> >>>>>> 2018년 7월 31일 (화) 오전 11:38, Holden Karau <hol...@pigscanfly.ca>님이 작성: >>>>>> >>>>>>> Th configuration file is optional, is there something you want to >>>>>>> try and change? >>>>>>> >>>>>>> On Mon, Jul 30, 2018 at 7:30 PM Hyukjin Kwon <gurwls...@gmail.com> >>>>>>> wrote: >>>>>>> >>>>>>>> I see. Thanks. I was wondering if I can see the configuration file >>>>>>>> since that looks needed (https://github.com/holdenk/me >>>>>>>> ntion-bot#configuration) but I couldn't find (sorry if it's just >>>>>>>> something I simply missed). >>>>>>>> >>>>>>>> 2018년 7월 31일 (화) 오전 1:48, Holden Karau <hol...@pigscanfly.ca>님이 작성: >>>>>>>> >>>>>>>>> So the one that is running is the the form in my own repo (set up >>>>>>>>> for K8s deployment) - http://github.com/holdenk/mention-bot >>>>>>>>> >>>>>>>>> On Mon, Jul 30, 2018 at 3:15 AM Hyukjin Kwon <gurwls...@gmail.com> >>>>>>>>> wrote: >>>>>>>>> >>>>>>>>>> Holden, so, is it a fork in https://github.com/facebook >>>>>>>>>> archive/mention-bot? Would you mind if I ask where I can see the >>>>>>>>>> configurations for it? >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> 2018년 7월 23일 (월) 오전 10:16, Holden Karau <hol...@pigscanfly.ca>님이 >>>>>>>>>> 작성: >>>>>>>>>> >>>>>>>>>>> Yeah so the issue with codeowners is it will only assign to >>>>>>>>>>> committers on the repo (the Beam project found this out the >>>>>>>>>>> practical >>>>>>>>>>> application way). >>>>>>>>>>> >>>>>>>>>>> I have a fork of mention bot running and it seems we can add it >>>>>>>>>>> (need an infra ticket), but one of the things the Beam folks asked >>>>>>>>>>> was to >>>>>>>>>>> not ping code authors who haven’t committed in the past year which >>>>>>>>>>> I need >>>>>>>>>>> to do a bit of poking on to make happen. >>>>>>>>>>> >>>>>>>>>>> On Sun, Jul 22, 2018 at 7:04 PM Nicholas Chammas < >>>>>>>>>>> nicholas.cham...@gmail.com> wrote: >>>>>>>>>>> >>>>>>>>>>>> On this topic, I just stumbled on a GitHub feature called >>>>>>>>>>>> CODEOWNERS <https://help.github.com/articles/about-codeowners/>. >>>>>>>>>>>> It lets you specify owners of specific areas of the repository >>>>>>>>>>>> using the >>>>>>>>>>>> same syntax that .gitignore uses. Here is CPython's CODEOWNERS >>>>>>>>>>>> file >>>>>>>>>>>> <https://github.com/python/cpython/blob/master/.github/CODEOWNERS> >>>>>>>>>>>> for reference. >>>>>>>>>>>> >>>>>>>>>>>> Dunno if that would complement mention-bot (which Facebook is >>>>>>>>>>>> apparently no longer maintaining >>>>>>>>>>>> <https://github.com/facebookarchive/mention-bot#readme>), or >>>>>>>>>>>> if we can even use it given the ASF setup on GitHub. But I thought >>>>>>>>>>>> it would >>>>>>>>>>>> be worth mentioning nonetheless. >>>>>>>>>>>> >>>>>>>>>>>> On Sat, Jul 14, 2018 at 11:17 AM Holden Karau < >>>>>>>>>>>> hol...@pigscanfly.ca> wrote: >>>>>>>>>>>> >>>>>>>>>>>>> Hearing no objections (and in a shout out to @ Nicholas >>>>>>>>>>>>> Chammas who initially suggested mention-bot back in 2016) I've >>>>>>>>>>>>> set up a >>>>>>>>>>>>> copy of mention bot and run it against my own repo (looks like >>>>>>>>>>>>> https://github.com/holdenk/spark-testing-base/pull/253 ). >>>>>>>>>>>>> >>>>>>>>>>>>> If no one objects I’ll ask infra to turn this on for Spark on >>>>>>>>>>>>> a trial biases and we can revisit it based on how folks interact >>>>>>>>>>>>> with it. >>>>>>>>>>>>> >>>>>>>>>>>>> On Wed, Jun 6, 2018 at 12:24 PM, Holden Karau < >>>>>>>>>>>>> hol...@pigscanfly.ca> wrote: >>>>>>>>>>>>> >>>>>>>>>>>>>> So there are a few bots along this line in OSS. If no one >>>>>>>>>>>>>> objects I’ll take a look and find one which matches our use case >>>>>>>>>>>>>> and try it >>>>>>>>>>>>>> out. >>>>>>>>>>>>>> >>>>>>>>>>>>>> On Wed, Jun 6, 2018 at 10:33 AM Sean Owen <sro...@gmail.com> >>>>>>>>>>>>>> wrote: >>>>>>>>>>>>>> >>>>>>>>>>>>>>> Certainly I will frequently dig through 'git blame' to >>>>>>>>>>>>>>> figure out who might be the right reviewer. Maybe that's >>>>>>>>>>>>>>> automatable -- >>>>>>>>>>>>>>> ping the person who last touched the most lines touched by the >>>>>>>>>>>>>>> PR? There >>>>>>>>>>>>>>> might be some false positives there. And I suppose the downside >>>>>>>>>>>>>>> is being >>>>>>>>>>>>>>> pinged forever for some change that just isn't well considered >>>>>>>>>>>>>>> or one of >>>>>>>>>>>>>>> those accidental 100K-line PRs. So maybe some way to decline or >>>>>>>>>>>>>>> silence is >>>>>>>>>>>>>>> important, or maybe just ping once and leave it. Sure, a bot >>>>>>>>>>>>>>> that just adds >>>>>>>>>>>>>>> a "Would @foo like to review?" comment on Github? Sure seems >>>>>>>>>>>>>>> worth trying >>>>>>>>>>>>>>> if someone is willing to do the work to cook up the bot. >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> On Wed, Jun 6, 2018 at 12:22 PM Holden Karau < >>>>>>>>>>>>>>> hol...@pigscanfly.ca> wrote: >>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> Hi friends, >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> Was chatting with some folks at the summit and I was >>>>>>>>>>>>>>>> wondering how people would feel about adding a review bot to >>>>>>>>>>>>>>>> ping folks. We >>>>>>>>>>>>>>>> already have the review dashboard but I was thinking we could >>>>>>>>>>>>>>>> ping folks >>>>>>>>>>>>>>>> who were the original authors of the code being changed whom >>>>>>>>>>>>>>>> might not be >>>>>>>>>>>>>>>> in the habit of looking at the review dashboard. >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> Cheers, >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> Holden :) >>>>>>>>>>>>>>>> -- >>>>>>>>>>>>>>>> Twitter: https://twitter.com/holdenkarau >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>> -- >>>>>>>>>>>>>> Twitter: https://twitter.com/holdenkarau >>>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>> -- >>>>>>>>>>>>> Twitter: https://twitter.com/holdenkarau >>>>>>>>>>>>> -- >>>>>>>>>>>>> Twitter: https://twitter.com/holdenkarau >>>>>>>>>>>>> >>>>>>>>>>>> -- >>>>>>>>>>> Twitter: https://twitter.com/holdenkarau >>>>>>>>>>> >>>>>>>>>> -- >>>>>>> Twitter: https://twitter.com/holdenkarau >>>>>>> >>>>>> > > > -- > Twitter: https://twitter.com/holdenkarau > -- Twitter: https://twitter.com/holdenkarau
