Skip to content

fix: grant SgtPooki admin on ipfs2foc - #55

Merged
jennijuju merged 1 commit into
FilOzone:masterfrom
SgtPooki:sgtpooki/admin-ipfs2foc
Jul 31, 2026
Merged

fix: grant SgtPooki admin on ipfs2foc#55
jennijuju merged 1 commit into
FilOzone:masterfrom
SgtPooki:sgtpooki/admin-ipfs2foc

Conversation

@SgtPooki

@SgtPooki SgtPooki commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Moves SgtPooki from push to admin on ipfs2foc.

I'm the primary maintainer. Push is enough to merge, not enough to administer: main is protected by a ruleset with no bypass actors, so correcting anything already merged needs an org admin. Ran into that today fixing a commit message that landed on main.

Already admin on atomicmemory.ai-demo, dealbot, and team-skills.

@github-actions

Copy link
Copy Markdown
Contributor

The following access changes will be introduced as a result of applying the plan:

Access Changes
User sgtpooki:
  - will have the permission to ipfs2foc change from push to admin

@github-actions

Copy link
Copy Markdown
Contributor

Before merge, verify that all the following plans are correct. After merge, Apply will regenerate the plans from the merged commit and continue only if they match.

Terraform plans

FilOzone
Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
-/+ destroy and then create replacement

Terraform will perform the following actions:

  # github_repository_collaborator.this["ipfs2foc:sgtpooki"] must be replaced
-/+ resource "github_repository_collaborator" "this" {
      ~ id                          = "ipfs2foc:SgtPooki" -> (known after apply)
      + invitation_id               = (known after apply)
      ~ permission                  = "push" -> "admin" # forces replacement
      + permission_diff_suppression = false
        # (2 unchanged attributes hidden)
    }

Plan: 1 to add, 0 to change, 1 to destroy.

I'm the primary maintainer. Push is enough to merge, not enough to
administer: main is protected by a ruleset with no bypass actors, so
correcting anything already merged needs an org admin.

Already admin on atomicmemory.ai-demo, dealbot, and team-skills.
@SgtPooki
SgtPooki force-pushed the sgtpooki/admin-ipfs2foc branch from 4845660 to 93d23e4 Compare July 31, 2026 16:29
@SgtPooki SgtPooki moved this from 📌 Triage to 🔎 Awaiting review in FOC Jul 31, 2026
@SgtPooki SgtPooki changed the title feat: SgtPooki admin on ipfs2foc fix: grant SgtPooki admin on ipfs2foc Jul 31, 2026
@github-project-automation github-project-automation Bot moved this from 🔎 Awaiting review to ✔️ Approved by reviewer in FOC Jul 31, 2026
@jennijuju
jennijuju merged commit 869ef5d into FilOzone:master Jul 31, 2026
6 checks passed
@github-project-automation github-project-automation Bot moved this from ✔️ Approved by reviewer to 🎉 Done in FOC Jul 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🎉 Done

Development

Successfully merging this pull request may close these issues.

3 participants