Skip to content

Fix for code scanning alert no. 4, 3 & 2: Workflow does not contain permissions - #8

Merged
antfroger merged 2 commits into
mainfrom
alert-autofix-4
Aug 19, 2026
Merged

Fix for code scanning alert no. 4, 3 & 2: Workflow does not contain permissions#8
antfroger merged 2 commits into
mainfrom
alert-autofix-4

Conversation

@antfroger

@antfroger antfroger commented Aug 19, 2026

Copy link
Copy Markdown
Owner

Potential fix for https://github.com/antfroger/progressive/security/code-scanning/2, https://github.com/antfroger/progressive/security/code-scanning/3, https://github.com/antfroger/progressive/security/code-scanning/4

Add an explicit permissions block to .github/workflows/lint.yaml at the workflow root so it applies to all jobs by default.
For this lint workflow, the least-privilege baseline is:

  • contents: read

This preserves existing functionality (checkout + lint dry-run) while satisfying CodeQL and least-privilege requirements. No imports, methods, or dependencies are needed since this is YAML configuration only.

Suggested fixes powered by Copilot Autofix. Review carefully before merging.

antfroger and others added 2 commits August 19, 2026 11:32
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@antfroger
antfroger marked this pull request as ready for review August 19, 2026 09:35
@antfroger antfroger changed the title Potential fix for code scanning alert no. 4: Workflow does not contain permissions Fix for code scanning alert no. 4, 3 & 2: Workflow does not contain permissions Aug 19, 2026
@antfroger
antfroger merged commit 7e6f8af into main Aug 19, 2026
19 checks passed
@antfroger
antfroger deleted the alert-autofix-4 branch August 19, 2026 09:37
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.

1 participant