Skip to content

Revisions#274

Merged
albertov19 merged 6 commits into
mainfrom
albertov19/pr-reviews-uninstall-permissions
Apr 9, 2026
Merged

Revisions#274
albertov19 merged 6 commits into
mainfrom
albertov19/pr-reviews-uninstall-permissions

Conversation

@albertov19

Copy link
Copy Markdown
Collaborator

Description

This pull request updates the Azure DevOps connection documentation to clarify the required permissions for connecting projects after admin consent. The main change specifies that only users with the Project Collection Administrator role can connect projects after admin consent, rather than all team members.

Documentation clarification:

  • Updated the note in code-reviews/pr-reviews/azure-devops.md to specify that after admin consent, only users with the Project Collection Administrator role (not all team members) can connect projects, and linked to the prerequisites section for more details.

Checklist

  • Required - I have added a label to this PR 🏷️
  • Required - I have run my changes through Grammarly
  • If pages have been moved, I have created redirects in the kluster-mkdocs repo

albertov19 and others added 3 commits April 9, 2026 16:39
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Enterprise note now appears before the tip, consistent with
GitHub, GitLab, Bitbucket, and Azure DevOps pages.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The note previously said all team members can connect projects after
admin consent, contradicting the Project Collection Administrator
requirement for webhook installation.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings April 9, 2026 14:42

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR clarifies Azure DevOps connection documentation by tightening who can connect projects after the one-time admin consent, aligning the guidance with required Azure DevOps permissions.

Changes:

  • Update the Azure DevOps admin-consent note to specify that only Project Collection-level admins can connect projects after consent.
  • Minor formatting-only adjustment to the Enterprise plan snippet.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 3 comments.

File Description
code-reviews/pr-reviews/azure-devops.md Updates the admin consent note to clarify post-consent connection permissions and links to prerequisites.
.snippets/text/code-reviews/pr-reviews-enterprise.md Formatting-only change in the Enterprise plan note snippet.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread code-reviews/pr-reviews/azure-devops.md Outdated
Comment thread code-reviews/pr-reviews/azure-devops.md Outdated
Comment thread code-reviews/pr-reviews/azure-devops.md Outdated
albertov19 and others added 3 commits April 9, 2026 14:48
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Fold permissions warning into the prerequisite bullet
- Fold custom rules note into the admin consent admonition
- Reduces page from 7 admonitions to 5

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@kluster-ai

kluster-ai Bot commented Apr 9, 2026

Copy link
Copy Markdown

🤖 kluster.ai PR Review Summary

Review this PR in 1/10 of the time — Try our new AI-powered code review experience

📋 PR Summary

This PR updates the documentation for setting up Azure DevOps with kluster.ai's PR Reviews feature. It clarifies that a member of Project Collection Administrators is required to generate a token and finish the setup, and provides additional guidance on the necessary permissions and troubleshooting steps if webhook installation fails. The changes also remove redundant warnings and consolidate information into the prerequisites section, making it easier for users to understand the requirements for a successful setup. Additionally, a note is added to mention that custom rules are not currently supported for Azure DevOps.

✅ All Clear

No issues detected in this PR.


⚠️ Warning: We haven't found any reviews performed on this branch from the IDE. Make sure to install the IDE and CLI integrations to catch issues earlier and accelerate your development workflow.

Powered by kluster.ai - Real-Time AI code review in your IDE

@albertov19 albertov19 merged commit cd27d9d into main Apr 9, 2026
6 of 7 checks passed
@albertov19 albertov19 deleted the albertov19/pr-reviews-uninstall-permissions branch April 9, 2026 14:57
@kluster-ai

kluster-ai Bot commented Apr 9, 2026

Copy link
Copy Markdown

🤖 kluster.ai PR Review Summary

Review this PR in 1/10 of the time — Try our new AI-powered code review experience

📋 PR Summary

This PR updates the documentation for setting up PR Reviews with Azure DevOps. It clarifies the required role for generating a personal access token and installing webhooks, specifying that a user must be a member of Project Collection Administrators. The changes reorganize and rephrase the content to improve clarity, moving information about the required role and permissions to the Prerequisites section and removing redundant warnings. Additionally, a note is added to indicate that custom rules are not currently supported for Azure DevOps. Overall, the changes aim to provide more concise and clear instructions for users setting up PR Reviews with Azure DevOps.

✅ All Clear

No issues detected in this PR.


⚠️ Warning: We haven't found any reviews performed on this branch from the IDE. Make sure to install the IDE and CLI integrations to catch issues earlier and accelerate your development workflow.

Powered by kluster.ai - Real-Time AI code review in your IDE

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.

2 participants