add comment clarifying maintain mapping

kerobbi · kerobbi · commit 4b5b6690aabd · 2026-04-23T11:41:26.000+01:00
diff --git a/pkg/lockdown/lockdown.go b/pkg/lockdown/lockdown.go
@@ -267,6 +267,8 @@ func (c *RepoAccessCache) checkPushAccess(ctx context.Context, username, owner,
 		return false, fmt.Errorf("failed to get user permission level: %w", err)
 	}
 
+	// REST API maps "maintain" to "write" (and "triage" to "read")
+	// https://docs.github.com/en/rest/collaborators/collaborators#get-repository-permissions-for-a-user
 	permission := permLevel.GetPermission()
 	return permission == "admin" || permission == "write", nil
 }

Original file line number	Diff line number	Diff line change
`@@ -267,6 +267,8 @@ func (c *RepoAccessCache) checkPushAccess(ctx context.Context, username, owner,`
`267`	`267`	`return false, fmt.Errorf("failed to get user permission level: %w", err)`
`268`	`268`	`}`
`269`	`269`
	`270`	`+ // REST API maps "maintain" to "write" (and "triage" to "read")`
	`271`	`+ // https://docs.github.com/en/rest/collaborators/collaborators#get-repository-permissions-for-a-user`
`270`	`272`	`permission := permLevel.GetPermission()`
`271`	`273`	`return permission == "admin" \|\| permission == "write", nil`
`272`	`274`	`}`