Skip to content

Make Copilot auth docs easier to scan - #54337

Merged
pelikhan merged 2 commits into
mainfrom
copilot/deep-report-split-dense-auth-sentence
Aug 20, 2026
Merged

Make Copilot auth docs easier to scan#54337
pelikhan merged 2 commits into
mainfrom
copilot/deep-report-split-dense-auth-sentence

Conversation

Copilot AI commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

The Copilot engine docs compressed two mutually exclusive authentication paths into one dense sentence. This made CI/CD setup harder to scan and increased the chance of choosing the wrong auth path.

  • Authentication guidance
    • Keeps the default engine selection note intact.
    • Splits Copilot authentication into two explicit branches:
      • organization-billed usage via copilot-requests: write
      • PAT-based usage via COPILOT_GITHUB_TOKEN
Authenticate Copilot in one of these ways:

- For organization-billed usage, grant `copilot-requests: write`.
- Otherwise, provide a `COPILOT_GITHUB_TOKEN` secret containing a fine-grained PAT with Copilot Requests access.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Update copilot.md to split dense auth sentence into scannable list Make Copilot auth docs easier to scan Aug 20, 2026
Copilot AI requested a review from pelikhan August 20, 2026 20:00
@pelikhan
pelikhan marked this pull request as ready for review August 20, 2026 20:08
Copilot AI balanced review requested due to automatic review settings August 20, 2026 20:08
@pelikhan
pelikhan merged commit 79f938d into main Aug 20, 2026
@pelikhan
pelikhan deleted the copilot/deep-report-split-dense-auth-sentence branch August 20, 2026 20:08

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

Improves Copilot authentication documentation readability and addresses #54314.

Changes:

  • Preserves the default engine guidance.
  • Separates organization-billed and PAT-based authentication into scannable list items.
Show a summary per file
File Description
docs/src/content/docs/engines/copilot.md Clarifies the two Copilot authentication paths.

Review details

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Balanced

@github-actions

Copy link
Copy Markdown
Contributor

🎉 This pull request is included in a new release.

Release: v0.87.2

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.

[deep-report] docs: split dense auth sentence in engines/copilot.md into scannable list

3 participants