Skip to content

ci(linkchecker): declare contents: read#130

Open
arpitjain099 wants to merge 1 commit into
cncf:mainfrom
arpitjain099:ci/add-permissions
Open

ci(linkchecker): declare contents: read#130
arpitjain099 wants to merge 1 commit into
cncf:mainfrom
arpitjain099:ci/add-permissions

Conversation

@arpitjain099
Copy link
Copy Markdown

Pins the default GITHUB_TOKEN for the only workflow here. lycheeverse/lychee-action uses the token solely to raise lychee's github.com rate limit (read-only API), so contents: read is sufficient.

YAML validated locally.

lychee uses GITHUB_TOKEN to raise its github.com rate limit; the
workflow does not need write access.

Signed-off-by: arpitjain099 <arpitjain099@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant