Skip to content

Conversation

@lidel
Copy link
Member

@lidel lidel commented Jan 15, 2025

Summary

Adding @guillaumemichel to @ipfs/kubo-maintainers

Why do you need this?

Gui will be spending more time contributing to IPFS and Go Triage

What else do we need to know?

DRI: myself

Reviewer's Checklist

  • It is clear where the request is coming from (if unsure, ask)
  • All the automated checks passed
  • The YAML changes reflect the summary of the request
  • The Terraform plan posted as a comment reflects the summary of the request

@lidel lidel requested a review from a team as a code owner January 15, 2025 00:32
@lidel lidel requested review from a team and galargh January 15, 2025 00:32
@github-actions
Copy link
Contributor

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

Access Changes
User guillaumemichel:
  - will have the permission to boxo change from maintain to admin
  - will have the permission to go-test change from maintain to admin
  - will have the permission to ipfs-check change from maintain to admin
  - will have the permission to kubo change from maintain to admin
  - will have the permission to rainbow change from maintain to admin
  - will have the permission to someguy change from maintain to admin

@github-actions
Copy link
Contributor

Before merge, verify that all the following plans are correct. They will be applied as-is after the merge.

Terraform plans

ipfs

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:

  # github_team_membership.this["kubo maintainers:guillaumemichel"] will be created
  + resource "github_team_membership" "this" {
      + etag     = (known after apply)
      + id       = (known after apply)
      + role     = "member"
      + team_id  = "6744049"
      + username = "guillaumemichel"
    }

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

@lidel lidel merged commit 91f5ba7 into master Jan 15, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants