Skip to content

Add workflow permissions to file-consistency Actions workflow#39

Merged
cjeanner merged 1 commit intomainfrom
fix/actions-workflow-permissions
Mar 23, 2026
Merged

Add workflow permissions to file-consistency Actions workflow#39
cjeanner merged 1 commit intomainfrom
fix/actions-workflow-permissions

Conversation

@cjeanner
Copy link
Collaborator

@cjeanner cjeanner commented Mar 23, 2026

Summary

Adds explicit permissions: contents: read to the file-consistency workflow (.github/workflows/yamllint.yml) so GITHUB_TOKEN follows least privilege and satisfies CodeQL actions/missing-workflow-permissions.

Addresses GitHub code scanning alert #1 for this repository.

Test plan

  • CI: file-consistency workflow runs on this PR
  • After merge, confirm code scanning alert clears on main

Made with Cursor

Declare explicit contents: read on the file-consistency GitHub Actions
workflow so GITHUB_TOKEN uses least privilege and satisfies CodeQL
rule actions/missing-workflow-permissions.

Resolves the public code scanning alert for missing workflow
permissions metadata (alert #1) in this repository.

AI-Assistance: Cursor IDE, Agent mode, Composer model.
Collaboration: standard human/AI workflow; human review and approval
before commit.
Security: addresses GitHub CodeQL finding on CI workflow permissions;
no production secrets or credentials involved.

Made-with: Cursor
@cjeanner cjeanner enabled auto-merge (squash) March 23, 2026 08:18
@cjeanner cjeanner self-assigned this Mar 23, 2026
Copy link

@abays abays left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@cjeanner cjeanner merged commit 61cd3ac into main Mar 23, 2026
1 check passed
@cjeanner cjeanner deleted the fix/actions-workflow-permissions branch March 23, 2026 08:48
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