Support mixing system aliases and names in the same command

Signed-off-by: Ihar Hrachyshka <[email protected]>

Author: booxter

README.md

@@ -355,7 +355,7 @@ since the architecture/operating system mismatches.
 By default, `nixpkgs-review` targets only the current system
 (`--systems current`). You can also explicitly provide one or several systems to
 target (`--systems "x86_64-linux aarch64-darwin"`). We also provide aliases for
-the flag:
+the flag, which can also be mixed with the system names.
 
 | Alias                                                | Transforms to                                             |
 | ---------------------------------------------------- | --------------------------------------------------------- |

nixpkgs_review/review.py

@@ -1,5 +1,6 @@
 import argparse
 import concurrent.futures
+import itertools
 import os
 import subprocess
 import sys
@@ -129,16 +130,10 @@ def __init__(
         self.skip_packages = skip_packages
         self.skip_packages_regex = skip_packages_regex
         self.local_system = current_system()
-        match len(systems):
-            case 0:
-                msg = "Systems is empty"
-                raise NixpkgsReviewError(msg)
-            case 1:
-                self.systems = self._process_aliases_for_systems(
-                    next(iter(systems)).lower()
-                )
-            case _:
-                self.systems = set(systems)
+        if not systems:
+            msg = "Systems is empty"
+            raise NixpkgsReviewError(msg)
+        self.systems = set(itertools.chain(*[self._process_aliases_for_systems(s.lower()) for s in systems]))
         self.allow = allow
         self.sandbox = sandbox
         self.build_graph = build_graph