fix(branch_protections.py): Sort Required Status Checks (#38)

Prevents false listing of differences when only order is different
andrewthetechie · May 25, 2023 · 4d4c44f · 4d4c44f
1 parent ca50c71
commit 4d4c44f
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/repo_manager/github/branch_protections.py b/repo_manager/github/branch_protections.py
@@ -315,6 +315,11 @@ def check_repo_branch_protections(
                     this_protection.required_status_checks.strict,
                 )
             )
+            # Without sorting, they sometimes get flagged as different just due to the ordinality of them
+            if (config_bp.protection.required_status_checks.checks is not None):
+                config_bp.protection.required_status_checks.checks.sort()
+            if (this_protection.required_status_checks.contexts is not None):
+                this_protection.required_status_checks.contexts.sort()
             diffs.append(
                 diff_option(
                     "required_status_checks::checks",