search

jasonkuster / merge-bot

An automatic merge robot for the ASF.
http://mergebot-vm.apache.org:8080
Apache License 2.0
2 stars 5 forks source link

Refresh authorized users every 15s. #22

Closed jasonkuster closed 7 years ago

jasonkuster commented 7 years ago

Signed-off-by: Jason Kuster jasonkuster@google.com

jasonkuster commented 7 years ago

re 15s frequency, that's the frequency at which we poll already so this was the easiest thing to add. I could add a timer, but it seems to me that if we wait too much longer than 15s people may get antsy waiting for the auth to propagate.

re subclass, sounds fine; I'd done it because I wasn't sure if we were just going to do git way back when we started, but it seems like that's the plan so I can clean that up. I'll take that as an AI for some future work.