Skip to content

Commit a53a87e

Browse files
committed
doc: remove plan permissions for GitHub app
Remove plan permissions from the organization permissions as it's not required. Signed-off-by: Chmouel Boudjnah <[email protected]>
1 parent 476170f commit a53a87e

File tree

3 files changed

+8
-8
lines changed

3 files changed

+8
-8
lines changed

docs/content/docs/install/github_apps.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,6 @@ Alternatively, you could set up manually by following the steps [here](./#setup-
4242

4343
* Select the following organization permissions:
4444
* **Members**: `Readonly`
45-
* **Plan**: `Readonly`
4645

4746
* Subscribe to following events:
4847
* Check run

pkg/cmd/tknpac/bootstrap/github.go

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -27,13 +27,12 @@ func generateManifest(opts *bootstrapOpts) ([]byte, error) {
2727
"push",
2828
},
2929
DefaultPermissions: &github.InstallationPermissions{
30-
Checks: github.Ptr("write"),
31-
Contents: github.Ptr("write"),
32-
Issues: github.Ptr("write"),
33-
Members: github.Ptr("read"),
34-
Metadata: github.Ptr("read"),
35-
OrganizationPlan: github.Ptr("read"),
36-
PullRequests: github.Ptr("write"),
30+
Checks: github.Ptr("write"),
31+
Contents: github.Ptr("write"),
32+
Issues: github.Ptr("write"),
33+
Members: github.Ptr("read"),
34+
Metadata: github.Ptr("read"),
35+
PullRequests: github.Ptr("write"),
3736
},
3837
}
3938
return json.Marshal(sc)

test/github_pullrequest_retest_test.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,8 @@ import (
1818
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
1919
)
2020

21+
// TestGithubSecondPullRequestGitopsCommentRetest will test the retest
22+
// functionality of a GitHub pull request.
2123
func TestGithubSecondPullRequestGitopsCommentRetest(t *testing.T) {
2224
if os.Getenv("NIGHTLY_E2E_TEST") != "true" {
2325
t.Skip("Skipping test since only enabled for nightly")

0 commit comments

Comments
 (0)