Skip to content

Add permissions to generate preview links - #55000

Merged
gewarren merged 1 commit into
mainfrom
gewarren-patch-8
Jul 22, 2026
Merged

Add permissions to generate preview links#55000
gewarren merged 1 commit into
mainfrom
gewarren-patch-8

Conversation

@gewarren

@gewarren gewarren commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator

Suggested by code scanning.


Internal previews

📄 File 🔗 Preview link
.github/workflows/generate-preview-links.yml .github/workflows/generate-preview-links

Add permissions for reading contents in the workflow.
Copilot AI review requested due to automatic review settings July 22, 2026 19:48
@gewarren
gewarren requested a review from a team as a code owner July 22, 2026 19:48
@dotnetrepoman dotnetrepoman Bot added this to the July 2026 milestone Jul 22, 2026
@gewarren
gewarren enabled auto-merge (squash) July 22, 2026 19:50

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 pull request updates the preview-link generator GitHub Actions workflow to explicitly set GITHUB_TOKEN permissions, as recommended by code scanning.

Changes:

  • Adds workflow-level permissions with contents: read for .github/workflows/generate-preview-links.yml.

Comment thread .github/workflows/generate-preview-links.yml
Comment thread .github/workflows/generate-preview-links.yml

@wadepickett wadepickett 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.

Approved assuming the existing issue is addressed.

@gewarren
gewarren merged commit 872d137 into main Jul 22, 2026
13 checks passed
@gewarren
gewarren deleted the gewarren-patch-8 branch July 22, 2026 21:26
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.

3 participants