-
Notifications
You must be signed in to change notification settings - Fork 4
Add PR Reviews nav, permissions warnings, uninstall commands, and rules #273
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
Changes from all commits
522e418
2480ee7
84260fa
dcef64e
c235047
b63b80c
720aa34
32195e6
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,2 @@ | ||
| !!! note "Enterprise plan" | ||
| PR Reviews is available exclusively on the Enterprise plan. To enable PR Reviews for your organization, contact the kluster.ai team at [sales@kluster.ai](mailto:sales@kluster.ai){target=\_blank}. |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -12,6 +12,8 @@ Before connecting, an administrator in your Azure DevOps organization must compl | |
|
|
||
| Once connected, the bot reviews every new pull request and every new commit pushed to an open pull request. No additional configuration is needed. | ||
|
|
||
| --8<-- 'text/code-reviews/pr-reviews-enterprise.md' | ||
|
|
||
| --8<-- 'text/code-reviews/pr-reviews-tip.md' | ||
|
|
||
| !!! warning "Custom rules not supported" | ||
|
|
@@ -22,9 +24,12 @@ Once connected, the bot reviews every new pull request and every new commit push | |
| Before getting started, ensure you have: | ||
|
|
||
| - A [kluster.ai](https://platform.kluster.ai/signup){target=\_blank} account. | ||
| - An Azure DevOps account with at least **Basic** access to the projects you want to review. | ||
| - An Azure DevOps account that is a member of **Project Collection Administrators** in your organization. | ||
| - A personal access token with the required scopes. See [Create a personal access token](#create-a-personal-access-token) for instructions. | ||
|
|
||
| !!! warning "Verify account permissions" | ||
| The account that generates the personal access token must be a member of **Project Collection Administrators**. Having the correct token scopes is not enough — the account itself needs this role to install webhooks. If the account has insufficient permissions, webhook installation will fail silently and pull request reviews will not appear. To fix this, navigate to **Organization Settings** > **Security** > **Permissions**, find the user, and add them to **Project Collection Administrators**. After updating permissions, click **Re-install** on the PR Reviews page in the kluster.ai platform to complete the setup. | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🤖 kluster.ai · 🟡 semantic · medium Description: The new requirement for Explanation: The new warning explicitly requires the user connecting the project to be a Recommended Action: The "Admin consent" section should be updated to clarify that only Project Collection Administrators can connect projects, replacing the claim about all team members. Issue Actions: |
||
|
|
||
| ## Admin consent | ||
|
|
||
| Before anyone in your organization can connect kluster to Azure DevOps, an administrator or organization owner must grant consent. This is a one-time step per organization that applies to both PR reviews and repo reviews. | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.