Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove web3-bot from codec-fixtures #82

Merged
merged 1 commit into from
May 9, 2024

Conversation

rvagg
Copy link
Member

@rvagg rvagg commented May 8, 2024

turns out this is neither useful nor does it fully work on this repo, so we'll just exclude it

turns out this is neither useful nor does it fully work on this repo, so we'll just exclude it
@rvagg rvagg requested a review from vmx May 8, 2024 05:16
@rvagg rvagg requested review from a team as code owners May 8, 2024 05:16
Copy link
Contributor

github-actions bot commented May 8, 2024

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

Access Changes
User web3-bot:
  - will lose push permission to codec-fixtures

Copy link
Contributor

github-actions bot commented May 8, 2024

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

Terraform plans

ipld

Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
  - destroy

Terraform will perform the following actions:

  # github_repository_collaborator.this["codec-fixtures:web3-bot"] will be destroyed
  # (because key ["codec-fixtures:web3-bot"] is not in for_each map)
  - resource "github_repository_collaborator" "this" {
      - id         = "codec-fixtures:web3-bot" -> null
      - permission = "push" -> null
      - repository = "codec-fixtures" -> null
      - username   = "web3-bot" -> null
    }

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

@rvagg
Copy link
Member Author

rvagg commented May 9, 2024

@vmx @galargh can one of you 👍 this please so I can make web3-bot stop opening PRs to the codec-fixtures repo?

@willscott willscott merged commit 5a06472 into master May 9, 2024
6 checks passed
@willscott willscott deleted the rvagg/codec-fixtures-no-web3 branch May 9, 2024 06:01
@galargh
Copy link
Contributor

galargh commented May 9, 2024

This makes sense 👍 @rvagg if you wanted to talk about how https://github.com/ipdxco/unified-github-workflows could better support repos like codec-fixtures or how we could help you with it in a custom way, let me know :)

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.

3 participants