about summary refs log tree commit diff
path: root/fun/clbot
AgeCommit message (Collapse)AuthorFilesLines
2024-05-27 r/8177 fix(fun/clbot): address sprintf warningFlorian Klink1-1/+1
%q is only defined for maps containing booleans, not strings. It should be OK to use %v here. Change-Id: I7475eb89032653a588ecdf70809eb35bbe7e04ab Reviewed-on: https://cl.tvl.fyi/c/depot/+/11730 Reviewed-by: tazjin <tazjin@tvl.su> Tested-by: BuildkiteCI Autosubmit: flokli <flokli@flokli.de>
2022-01-28 r/3694 feat(clbot): Add list of strings that should never pingVincent Ambo1-0/+8
... including marcus, which is the name of a person in the channel as well as the name of a machine of wpcarro's. Change-Id: I84bd83413175880663b3dafbdb760bbe9d23cef5 Reviewed-on: https://cl.tvl.fyi/c/depot/+/5086 Tested-by: BuildkiteCI Reviewed-by: sterni <sternenseemann@systemli.org> Autosubmit: tazjin <tazjin@tvl.su>
2021-12-14 r/3239 fix(clbot): Noping autosubmitted messageGriffin Smith1-5/+1
Change-Id: I6b7e086d2f10751f2f8b8c90eded01edcbfabadb Reviewed-on: https://cl.tvl.fyi/c/depot/+/4317 Tested-by: BuildkiteCI Reviewed-by: tazjin <mail@tazj.in> Autosubmit: grfn <grfn@gws.fyi>
2021-12-10 r/3186 fix(clbot): Use change *owner* and not *uploader*Vincent Ambo1-6/+6
In autosubmit cases that require rebases, the change *uploader* might be clbot which would cause besadii to use clbot as the owner. This is incorrect, but luckily the change-merged event has an actual owner field instead. Change-Id: Ia35b52085f94628e61eb358807b3b85565521b60
2021-12-10 r/3179 feat(fun/clbot): Add distinct messages for auto-submitted CLsVincent Ambo1-2/+13
Detects autosubmitted CLs (other people's CLs submitted by clbot) and modifies the text submitted to IRC accordingly. If a CL is autosubmitted, we opt to highlight its author rather than invoking noping. Change-Id: Ibc21b7eeb2f0f2087097404baef6976384d68b09
2021-05-22 r/2609 chore: Replace Freenode mentions with HackIntVincent Ambo1-2/+2
This doesn't replace all of them in the repo, but at least the ones that are relevant to our move. Change-Id: I842e7594b4c16af30d880272417874f6b29afd22 Reviewed-on: https://cl.tvl.fyi/c/depot/+/3134 Tested-by: BuildkiteCI Reviewed-by: lukegb <lukegb@tvl.fyi> Reviewed-by: grfn <grfn@gws.fyi>
2021-05-22 r/2603 feat(clbot): Add a flag to disable TLS connectionsVincent Ambo1-2/+14
The local bouncer on whitby does not use TLS. Change-Id: Idf9c56f94129b0ddce620eb559082a8f2f088078 Reviewed-on: https://cl.tvl.fyi/c/depot/+/3128 Tested-by: BuildkiteCI Reviewed-by: grfn <grfn@gws.fyi>
2021-04-02 r/2410 chore(clbot): reformat messagesLuke Granger-Brown1-2/+2
Instead of putting the subject in quotes, we instead reorder the message to be of the format: CL/2768 applied by lukegb - chore(clbot): reformat messages - https://cl.tvl.fyi/2768 Change-Id: I77d19525c399396a91797f423bdfc92069d47f9f Reviewed-on: https://cl.tvl.fyi/c/depot/+/2768 Reviewed-by: glittershark <grfn@gws.fyi> Tested-by: BuildkiteCI
2021-03-28 r/2352 fix(clbot): Avoid pinging users for their own user folder CLsVincent Ambo1-2/+12
Instead of only "nopinging" the username in the templated message, replace all instances of the CL owner's name with one that does not (tries to not) highlight them. This way, CLs sent to another user's folder will still highlight them. Change-Id: I9a3d8563ab32befc1a1b1412851026343c170dd3 Reviewed-on: https://cl.tvl.fyi/c/depot/+/2688 Tested-by: BuildkiteCI Reviewed-by: sterni <sternenseemann@systemli.org> Reviewed-by: glittershark <grfn@gws.fyi> Reviewed-by: lukegb <lukegb@tvl.fyi>
2020-07-02 r/1161 fix(clbot): s/submitted/applied/isomer1-1/+1
By vote of the IRC channel. Change-Id: I5ea965d45a7d73b0bdcae817d16eaa17e47adcbc Reviewed-on: https://cl.tvl.fyi/c/depot/+/861 Reviewed-by: isomer <isomer@tvl.fyi> Reviewed-by: glittershark <grfn@gws.fyi> Reviewed-by: lukegb <lukegb@tvl.fyi> Reviewed-by: BuildkiteCI Tested-by: BuildkiteCI
2020-06-24 r/1075 feat(clbot): allow specifying branch and repo to look at on command lineLuke Granger-Brown1-2/+26
Change-Id: Ib2e37275d770a1e98a526481018514a564a4f73b Reviewed-on: https://cl.tvl.fyi/c/depot/+/569 Reviewed-by: tazjin <mail@tazj.in>
2020-06-16 r/1001 fix(clbot): actually display people's usernames properlyLuke Granger-Brown1-1/+2
Change-Id: Idcbb6ffbb3127520141596cea7883cd137a1ea56 Reviewed-on: https://cl.tvl.fyi/c/depot/+/441 Reviewed-by: eta <eta@theta.eu.org> Reviewed-by: riking <rikingcoding@gmail.com>
2020-06-16 r/992 feat(clbot): insert Unicode ZWS to avoid pinging userseta1-4/+12
Change-Id: I732bf6daa01f5b2099c4f4cbdd21a2ceedc79cb1 Reviewed-on: https://cl.tvl.fyi/c/depot/+/385 Reviewed-by: q3k <q3k@q3k.org> Reviewed-by: lukegb <lukegb@tvl.fyi>
2020-06-16 r/990 feat(clbot): Add IRC support to the IRC bot.Luke Granger-Brown4-26/+158
Change-Id: I183488824882750c46e7216b98ab48e1d8f48096 Reviewed-on: https://cl.tvl.fyi/c/depot/+/343 Reviewed-by: eta <eta@theta.eu.org> Reviewed-by: tazjin <mail@tazj.in>
2020-06-15 r/963 chore(clbot): Refactor backoff utility into a separate package.Luke Granger-Brown4-28/+60
It'll be reused by the IRC side of things too. Change-Id: I3d84f3fd5fca6a6d948f331143b14f096d10675d Reviewed-on: https://cl.tvl.fyi/c/depot/+/342 Reviewed-by: tazjin <mail@tazj.in>
2020-06-15 r/962 chore(clbot): Add signal handler to make clbot shutdown cleanly on SIGINT.Luke Granger-Brown1-1/+24
Change-Id: I3c6eeeb99f9d81cdbcb10880c9075ac94c4f5d19 Reviewed-on: https://cl.tvl.fyi/c/depot/+/341 Reviewed-by: tazjin <mail@tazj.in>
2020-06-14 r/950 feat(clbot): Create Gerrit watcher and basic clbot binary.Luke Granger-Brown11-0/+1201
gerrit.Watcher is a class which watches the Gerrit stream-events SSH connection and produces events. There's a basic CLBot binary as well, to demonstrate driving it to produce messages on the logging output. It doesn't really do anything else. Change-Id: I274fe0a77c8329f79456425405e2fbdc3ca2edf0 Reviewed-on: https://cl.tvl.fyi/c/depot/+/245 Reviewed-by: tazjin <mail@tazj.in>