Skip to content

Commit ebf98b7

Browse files
committed
Remove mentions of OrganizationAdmin and actor_id 1
Signed-off-by: Timo Sand <timo.sand@f-secure.com>
1 parent ccad8e4 commit ebf98b7

4 files changed

Lines changed: 1 addition & 9 deletions

File tree

docs/resources/repository_ruleset.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -304,7 +304,6 @@ The `rules` block supports the following:
304304

305305
~> Note: at the time of writing this, the following actor types correspond to the following actor IDs:
306306

307-
- `OrganizationAdmin` -> `1`
308307
- `RepositoryRole` (This is the actor type, the following are the base repository roles and their associated IDs.)
309308
- `maintain` -> `2`
310309
- `write` -> `4`

github/resource_github_organization_ruleset_test.go

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,6 @@ resource "github_organization_ruleset" "test" {
7272
}
7373
7474
bypass_actors {
75-
actor_id = 1
7675
actor_type = "OrganizationAdmin"
7776
bypass_mode = "always"
7877
}
@@ -167,7 +166,6 @@ resource "github_organization_ruleset" "test" {
167166
resource.TestCheckResourceAttr("github_organization_ruleset.test", "bypass_actors.1.actor_id", "5"),
168167
resource.TestCheckResourceAttr("github_organization_ruleset.test", "bypass_actors.1.actor_type", "RepositoryRole"),
169168
resource.TestCheckResourceAttr("github_organization_ruleset.test", "bypass_actors.1.bypass_mode", "always"),
170-
resource.TestCheckResourceAttr("github_organization_ruleset.test", "bypass_actors.2.actor_id", "1"),
171169
resource.TestCheckResourceAttr("github_organization_ruleset.test", "bypass_actors.2.actor_type", "OrganizationAdmin"),
172170
resource.TestCheckResourceAttr("github_organization_ruleset.test", "bypass_actors.2.bypass_mode", "always"),
173171
resource.TestCheckResourceAttr("github_organization_ruleset.test", "rules.0.pull_request.0.allowed_merge_methods.#", "3"),
@@ -596,7 +594,6 @@ resource "github_organization_ruleset" "test" {
596594
}
597595
598596
bypass_actors {
599-
actor_id = 1
600597
actor_type = "OrganizationAdmin"
601598
bypass_mode = "always"
602599
}
@@ -640,7 +637,6 @@ resource "github_organization_ruleset" "test" {
640637
resource.TestCheckResourceAttr("github_organization_ruleset.test", "bypass_actors.1.actor_id", "5"),
641638
resource.TestCheckResourceAttr("github_organization_ruleset.test", "bypass_actors.1.actor_type", "RepositoryRole"),
642639
resource.TestCheckResourceAttr("github_organization_ruleset.test", "bypass_actors.1.bypass_mode", "always"),
643-
resource.TestCheckResourceAttr("github_organization_ruleset.test", "bypass_actors.2.actor_id", "1"),
644640
resource.TestCheckResourceAttr("github_organization_ruleset.test", "bypass_actors.2.actor_type", "OrganizationAdmin"),
645641
resource.TestCheckResourceAttr("github_organization_ruleset.test", "bypass_actors.2.bypass_mode", "always"),
646642
resource.TestCheckResourceAttr("github_organization_ruleset.test", "rules.0.file_path_restriction.0.restricted_file_paths.0", "test.txt"),
@@ -723,7 +719,6 @@ resource "github_organization_ruleset" "test" {
723719
}
724720
725721
bypass_actors {
726-
actor_id = 1
727722
actor_type = "OrganizationAdmin"
728723
bypass_mode = "always"
729724
}
@@ -803,7 +798,6 @@ resource "github_organization_ruleset" "test" {
803798
enforcement = "active"
804799
805800
bypass_actors {
806-
actor_id = 1
807801
actor_type = "OrganizationAdmin"
808802
bypass_mode = "%s"
809803
}

github/resource_github_repository_ruleset.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ func resourceGithubRepositoryRuleset() *schema.Resource {
6868
Type: schema.TypeInt,
6969
Optional: true,
7070
Default: nil,
71-
Description: "The ID of the actor that can bypass a ruleset. When `actor_type` is `OrganizationAdmin`, this should be set to `1`. Some resources such as DeployKey do not have an ID and this should be omitted.",
71+
Description: "The ID of the actor that can bypass a ruleset. Some resources such as DeployKey do not have an ID and this should be omitted.",
7272
},
7373
"actor_type": {
7474
Type: schema.TypeString,

templates/resources/repository_ruleset.md.tmpl

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -233,7 +233,6 @@ The `rules` block supports the following:
233233

234234
~> Note: at the time of writing this, the following actor types correspond to the following actor IDs:
235235

236-
- `OrganizationAdmin` -> `1`
237236
- `RepositoryRole` (This is the actor type, the following are the base repository roles and their associated IDs.)
238237
- `maintain` -> `2`
239238
- `write` -> `4`

0 commit comments

Comments
 (0)