Skip to content

Fallback to backend for test roulette

Rémy Coutable requested to merge fallback-to-backend-for-test-roulette into master

What does this MR do?

Given that:

  • The :test category sometimes have no potential reviewers.
  • The :engineering_productivity category sometimes have no potential maintainers (e.g. when I'm opening a MR since only @godfat and me are :engineering_productivity maintainers but @godfat is currently OOO.

This MR fallback to pick a :backend reviewer/maintainer in those two cases. Note that if there's already a reviewer/maintainer picked for the :backend category, we'll re-use this one, otherwise a new one is picked.

Validation

backend and Engineering Productivity change but no maintainer available for Engineering Productivity => fallback to the backend maintainer

Screen_Shot_2020-04-02_at_16.27.04

backend and test/Quality change but no reviewer available for test/Quality => fallback to the backend reviewer

Screen_Shot_2020-04-02_at_17.44.53

Does this MR meet the acceptance criteria?

Conformity

Edited by Rémy Coutable

Merge request reports