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

fevm-contract-tests #62

Merged
merged 3 commits into from
Aug 28, 2024
Merged

fevm-contract-tests #62

merged 3 commits into from
Aug 28, 2024

Conversation

rvagg
Copy link
Member

@rvagg rvagg commented Aug 26, 2024

Mainly I just want access to this repo, to at least push, but since nobody else is caring for it, admin would be good to take care of some things. I've added lotus-maintainers and FIL-B here for now, that's my best guess for who should "own" it.

@rvagg rvagg changed the title Rvagg/fevm contract tests fevm-contract-tests Aug 26, 2024
Copy link
Contributor

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

Access Changes
User aarshkshah1992:
  - will gain admin permission to fevm-contract-tests
User arajasek:
  - will lose push permission to fevm-contract-tests
User biglep:
  - will gain admin permission to fevm-contract-tests
User jennijuju:
  - will gain admin permission to fevm-contract-tests
User kubuxu:
  - will have the permission to fevm-contract-tests change from push to admin
User longfeiwan9:
  - will gain admin permission to fevm-contract-tests
User maciejwitowski:
  - will lose push permission to fevm-contract-tests
User magik6k:
  - will gain admin permission to fevm-contract-tests
User masih:
  - will gain admin permission to fevm-contract-tests
User rjan90:
  - will gain admin permission to fevm-contract-tests
User rk-rishikesh:
  - will gain admin permission to fevm-contract-tests
User rvagg:
  - will gain admin permission to fevm-contract-tests
User snissn:
  - will gain admin permission to fevm-contract-tests
User stebalien:
  - will have the permission to fevm-contract-tests change from push to admin
User tippyflitsuk:
  - will gain admin permission to fevm-contract-tests
User trruckerfling:
  - will gain admin permission to fevm-contract-tests
User xbalbinus:
  - will gain admin permission to fevm-contract-tests
User xors-eng:
  - will gain admin permission to fevm-contract-tests
User zenground0:
  - will have the permission to fevm-contract-tests change from push to admin

Copy link
Contributor

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

Terraform plans

filecoin-project

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

Terraform will perform the following actions:

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

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

  # github_team_repository.this["fil-b:fevm-contract-tests"] will be created
  + resource "github_team_repository" "this" {
      + etag       = (known after apply)
      + id         = (known after apply)
      + permission = "admin"
      + repository = "fevm-contract-tests"
      + team_id    = "10353148"
    }

  # github_team_repository.this["lotus-maintainers:fevm-contract-tests"] will be created
  + resource "github_team_repository" "this" {
      + etag       = (known after apply)
      + id         = (known after apply)
      + permission = "admin"
      + repository = "fevm-contract-tests"
      + team_id    = "4890221"
    }

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

@rvagg rvagg merged commit fe4ac36 into master Aug 28, 2024
6 checks passed
@rvagg rvagg deleted the rvagg/fevm-contract-tests branch August 28, 2024 01:26
@rvagg rvagg mentioned this pull request Aug 28, 2024
4 tasks
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