diff --git a/.github/workflows/sync-users-check.yml b/.github/workflows/sync-users-check.yml index cfd152f..f47b647 100644 --- a/.github/workflows/sync-users-check.yml +++ b/.github/workflows/sync-users-check.yml @@ -5,8 +5,6 @@ on: pull_request: branches: - main - paths: - - 'users.yaml' jobs: sync-users-check: runs-on: ubuntu-latest diff --git a/manage-users.py b/manage-users.py index f445d47..28adf6f 100644 --- a/manage-users.py +++ b/manage-users.py @@ -160,7 +160,8 @@ def remove_ghidra_users(current_users,yaml_users,dry_run): print(ghidra_user, "isn't in the source control list anymore. Removing...",end='') if not dry_run: - remove_user(ghidra_user) + # Commenting this out for now + # remove_user(ghidra_user) print("Done.") else: print("Dry run mode. Skipping.") @@ -236,7 +237,7 @@ def add_ghidra_users(current_users,yaml_users,dry_run): print("Setting permissions for", ghidra_user["ghidraName"] + "...", end='') if not dry_run: - # check_and_set_permission(ghidra_user["ghidraName"], ghidra_user["permissions"]) + check_and_set_permission(ghidra_user["ghidraName"], ghidra_user["permissions"]) print("Done.") else: print("Dry run mode. Skipping.") @@ -266,6 +267,7 @@ def manage_users(dry_run=False): if dry_run: if rm_users > 0 or update_users > 0: + print("Users are being updated or deleted. Failing automerge check!") sys.exit(1) elif add_users["num_users"] > 0: for permission in add_users["permissions"]: