Skip to content

Set explicit least-privilege GITHUB_TOKEN permissions in CI workflow - #5

Merged
lisagorewitdecker merged 2 commits into
masterfrom
copilot/fix-code-scanning-alerts
Sep 8, 2026
Merged

lisagorewitdecker merged 2 commits into
masterfrom
copilot/fix-code-scanning-alerts

Conversation

Copilot AI commented Sep 8, 2026 •

Copy link
Copy Markdown

This PR addresses the code scanning finding for missing explicit workflow token permissions. It narrows workflow token scope by declaring least-privilege permissions at workflow level in CI.

  • What changed

    • Added explicit workflow permissions in .github/workflows/CI.yml:
      • contents: read
  • Why this resolves the alert

    • Satisfies CodeQL鈥檚 actions/missing-workflow-permissions requirement by avoiding implicit/default token scopes.
  • Patch excerpt

    permissions:
      contents: read

@lisagorewitdecker lisagorewitdecker changed the title [WIP] Fix code scanning alert(s) flagged in repository Fix code scanning alert(s) flagged in repository Sep 8, 2026
@lisagorewitdecker
lisagorewitdecker marked this pull request as ready for review September 8, 2026 13:36
Copilot AI lite review requested due to automatic review settings September 8, 2026 13:36

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.

Copilot wasn't able to review any files in this pull request.


馃挕 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Co-authored-by: lisagorewitdecker <50784106+lisagorewitdecker@users.noreply.github.com>
Copilot AI changed the title Fix code scanning alert(s) flagged in repository Set explicit least-privilege GITHUB_TOKEN permissions in CI workflow Sep 8, 2026
@lisagorewitdecker
lisagorewitdecker merged commit fd8451d into master Sep 8, 2026
5 of 6 checks passed
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