Skip to content

Conversation

@aschmahmann
Copy link
Contributor

Summary

Add myself to the multicodec repo

Why do you need this?

I used to have permissions here, but they were dropped (not sure why nobody tagged me) and I'm still involved and look at PRs as time allows. Some PRs like multiformats/multicodec#375 were not reviewed by anyone else with access.

This shouldn't net change my permissions but using the "override" didn't feel appropriate so flagging by adding myself to codeowners.

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

@aschmahmann aschmahmann requested review from a team as code owners May 19, 2025 17:02
@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

multiformats

Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
  ~ update in-place

Terraform will perform the following actions:

  # github_repository_file.this["multicodec/codeowners"] will be updated in-place
  ~ resource "github_repository_file" "this" {
      + commit_author       = "GitHub"
      + commit_email        = "noreply@github.com"
      ~ content             = <<-EOT
          - * @darobin @rvagg @vmx
          + * @darobin @rvagg @vmx @aschmahmann
        EOT
        id                  = "multicodec/CODEOWNERS"
        # (8 unchanged attributes hidden)
    }

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

@github-actions
Copy link
Contributor

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

Access Changes
There will be no access changes

@aschmahmann aschmahmann merged commit 0a3e03d into master May 20, 2025
6 checks passed
@aschmahmann aschmahmann deleted the multicodec-add-adin branch May 20, 2025 07:01
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.

4 participants