Skip to content

Add a permissions block to linkcheck.yml, matching the other workflows - #55

Merged
mmcky merged 2 commits into
mainfrom
linkcheck-permissions
Aug 6, 2026
Merged

Add a permissions block to linkcheck.yml, matching the other workflows#55
mmcky merged 2 commits into
mainfrom
linkcheck-permissions

Conversation

@mmcky

@mmcky mmcky commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Completes the permissions hardening #54 landed for ci, collab and publishlinkcheck.yml is the one remaining workflow using dawidd6/action-download-artifact without an explicit block. The download works today via public-repo API leniency (the same mechanism the sibling repos' ci workflows rely on), so this is consistency hardening rather than a live fix: contents: read + actions: read, with the same comment the other blocks carry.

Raised by Copilot's review on the superseded #53 (its other two comments were addressed by #54's consolidated blocks). With this merged, every workflow in the repo that touches the Actions API carries an explicit grant, and the repo-level token default can drop to read with nothing relying on leniency.

🤖 Generated with Claude Code

Completes the hardening #54 landed for ci, collab and publish: the
dawidd6 cache download works today via public-repo API leniency, but
granting actions: read explicitly matches the documented requirement
and the blocks the other workflows now carry. Raised by Copilot on
the superseded #53.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings August 5, 2026 03:24
@netlify

netlify Bot commented Aug 5, 2026

Copy link
Copy Markdown

Deploy Preview for timely-seahorse-68815c ready!

Name Link
🔨 Latest commit 59a01ef
🔍 Latest deploy log https://app.netlify.com/projects/timely-seahorse-68815c/deploys/6a73d1d4ed41b50008830087
😎 Deploy Preview https://deploy-preview-55--timely-seahorse-68815c.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copilot AI left a comment

Copy link
Copy Markdown

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 hardens the GitHub Actions linkcheck.yml workflow by adding an explicit job-level permissions block, aligning it with the repository’s other workflows that interact with the Actions API.

Changes:

  • Adds an explicit permissions: block to linkcheck.yml granting contents: read and actions: read.
  • Documents (inline) why actions: read is required for downloading the cache.yml artifact via dawidd6/action-download-artifact.

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

@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown

@mmcky
mmcky merged commit 10bac27 into main Aug 6, 2026
6 checks passed
@mmcky
mmcky deleted the linkcheck-permissions branch August 6, 2026 00:38
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