abcxyz / guardian

Apache License 2.0
17 stars 1 forks source link

fix: return only teams actor is a member of #432

Closed gjonathanhong closed 1 week ago

gjonathanhong commented 1 week ago

There was a bug that ended up returning all teams in the organization as the teams the github actor is a member of. This properly returns only the teams they are members of, both direct and inherited.

github-actions[bot] commented 1 week ago

🔱 Guardian 🔱

Guardian will run for the following directories

Directories terraform/project1 terraform/project2

github-actions[bot] commented 1 week ago

🔱 Guardian 🔱 PLAN 🟩 SUCCESS [logs]

Entrypoint: terraform/project1

Details ```diff Terraform used the selected providers to generate the following execution plan. Resource actions are indicated with the following symbols: + create Terraform will perform the following actions: # null_resource.empty will be created + resource "null_resource" "empty" { + id = (known after apply) } Plan: 1 to add, 0 to change, 0 to destroy. ```
github-actions[bot] commented 1 week ago

🔱 Guardian 🔱 PLAN 🟩 SUCCESS [logs]

Entrypoint: terraform/project2

Details ```diff Terraform used the selected providers to generate the following execution plan. Resource actions are indicated with the following symbols: + create Terraform will perform the following actions: # null_resource.empty will be created + resource "null_resource" "empty" { + id = (known after apply) } Plan: 1 to add, 0 to change, 0 to destroy. ```
github-actions[bot] commented 1 week ago

🔱 Guardian 🔱 APPLY 🟩 SUCCESS [logs]

Entrypoint: terraform/project2

Details ```diff null_resource.empty: Creating... null_resource.empty: Creation complete after 0s [id=1916248146859730466] Apply complete! Resources: 1 added, 0 changed, 0 destroyed. ```
github-actions[bot] commented 1 week ago

🔱 Guardian 🔱 APPLY 🟩 SUCCESS [logs]

Entrypoint: terraform/project1

Details ```diff null_resource.empty: Creating... null_resource.empty: Creation complete after 0s [id=777028529940722900] Apply complete! Resources: 1 added, 0 changed, 0 destroyed. ```
drevell commented 1 week ago

Just based on the PR description, this area seems in need of some unit tests.