Skip to content

Commit

Permalink
Note alongside TEAM_REX that changing it affects `DependabotRotaRep…
Browse files Browse the repository at this point in the history
…orter`, to avoid it happening without this being noticed.
  • Loading branch information
mikerkelly committed Jan 15, 2025
1 parent dbb755b commit 81be38c
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 0 deletions.
1 change: 1 addition & 0 deletions workspace/dependabot/jobs.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ class DependabotRotaReporter(RotaReporter):
The candidates are currently Team Rex (except Katie).
If the candidate definition or Team changes this will affect
the rota offset and the rota will restart at an arbitrary point.
Consider redesigning class to include an offset if that happens.
"""

def get_rota(self) -> dict:
Expand Down
4 changes: 4 additions & 0 deletions workspace/utils/people.py
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,10 @@ def by_github_username(self, github_username):
)


# Note that adding to or re-ordering this list will will affect the
# DependabotRotaReporter ordering algorithm, restarting it at an arbitrary
# point. If you need to change this list, consider redesigning that class to
# include an offset.
TEAM_REX = [
PEOPLE["Jon"],
PEOPLE["Katie"],
Expand Down

0 comments on commit 81be38c

Please sign in to comment.