fix: remove duplicate bypass_pull_request_allowances from suborgs

safe-settings deepMerge concatenates arrays, so having nbrieussel in
both settings.yml and each suborg file produced ["nbrieussel","nbrieussel"]
after merge. The NOP diff showed this as a deletion — meaning the next
real sync would have removed the bypass entirely.

Keeping bypass_pull_request_allowances only in settings.yml (org level).
The deepMerge carries it through to suborg repos since the suborg files
no longer override that key.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: 2 people

.github/suborgs/backend.yml

@@ -32,11 +32,7 @@ branches:
         dismiss_stale_reviews: true
         require_code_owner_reviews: false
         require_last_push_approval: false
-        bypass_pull_request_allowances:
-          apps: []
-          users:
-            - nbrieussel
-          teams: []
+
       required_status_checks:
         strict: true   # Branch must be up to date with main before merge
         contexts: []

.github/suborgs/frontend.yml

@@ -30,11 +30,7 @@ branches:
         dismiss_stale_reviews: true
         require_code_owner_reviews: false
         require_last_push_approval: false
-        bypass_pull_request_allowances:
-          apps: []
-          users:
-            - nbrieussel
-          teams: []
+
       required_status_checks:
         strict: true
         contexts: []

.github/suborgs/infrastructure.yml

@@ -31,11 +31,7 @@ branches:
         dismiss_stale_reviews: true
         require_code_owner_reviews: true   # CODEOWNERS file should designate platform team
         require_last_push_approval: true   # Last commit must be approved by someone else
-        bypass_pull_request_allowances:
-          apps: []
-          users:
-            - nbrieussel
-          teams: []
+
       required_status_checks:
         strict: true
         contexts: []