Move out_of_office? from reviewer roulette to Teammate class

Inactive Accountrequested to merge
winh-teammate-out-of-office into master

What does this MR do?

Improves Danger-based reviewer roulette by

  • moving http_get_json to new RequestHelper module.
  • moving out_of_office? to the already existing Teammate class.
  • caching the user status response (mostly relevant for Traintainers who are iterated over three times).
  • expecting a HTTP status of 200 (rather than 2xx).
Edited by Inactive Account

Merge request reports