This a-or-b-but-not-both seems a bit awkward. Maybe there should be a separate function, `should_backport_by_milestone`, so they can each take a single, required parameter.