Skip to content

Potential fix for code scanning alert no. 5: Workflow does not contain permissions#20

Merged
nstarman merged 1 commit intomainfrom
permissions-ci
Oct 1, 2025
Merged

Potential fix for code scanning alert no. 5: Workflow does not contain permissions#20
nstarman merged 1 commit intomainfrom
permissions-ci

Conversation

@nstarman
Copy link
Contributor

@nstarman nstarman commented Oct 1, 2025

Potential fix for https://github.com/GalacticDynamics/plotting_backends/security/code-scanning/5

To resolve this, you should add an explicit permissions block to either the root of the workflow (to apply to all jobs), or to each job (for finer granularity). The minimal sufficient permission for these jobs are likely to be only read access to repository contents: contents: read. No steps shown require writing to the repository, interacting with issues or pull requests, or other elevated actions, so write access does not need to be granted. The change should be made near the top of the workflow file (e.g., after the name: and before on:) for all jobs, or directly underneath the specific job(s) if only certain jobs need reduced permissions.

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

…n permissions

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Signed-off-by: Nathaniel Starkman <nstarman@users.noreply.github.com>
@nstarman nstarman added this to the v0.2.0 milestone Oct 1, 2025
@nstarman nstarman marked this pull request as ready for review October 1, 2025 16:55
@nstarman nstarman merged commit a2c3c67 into main Oct 1, 2025
15 checks passed
@nstarman nstarman deleted the permissions-ci branch October 1, 2025 16:55
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

Comments