Perms fix (#78)

* enable setting of perms

* test dry run fix

* add more verbose logging

* rm test user

.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

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"]: