FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Potential fix for code scanning alert no. 1: Workflow does not contain permissions by SemyonSinchenko · Pull Request #832 · graphframes/graphframes · GitHub

Potential fix for code scanning alert no. 1: Workflow does not contain permissions - #832

Merged
SemyonSinchenko merged 1 commit into
mainfrom
alert-autofix-1
May 1, 2026
Merged

Potential fix for code scanning alert no. 1: Workflow does not contain permissions#832
SemyonSinchenko merged 1 commit into
mainfrom
alert-autofix-1

Conversation

Copy link
Copy Markdown
Collaborator

Potential fix for https://github.com/graphframes/graphframes/security/code-scanning/1

Add an explicit permissions block at the workflow root so all jobs inherit least-privilege defaults.
For this workflow, the best minimal non-breaking starting point is:

  • contents: read

This supports actions/checkout and typical read-only CI operations while preventing unnecessary write scopes.

Where to change: .github/workflows/scala-ci.yml, directly after the on: declaration and before jobs:.

No imports, methods, or dependency changes are needed.

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>
SemyonSinchenko marked this pull request as ready for review April 30, 2026 11:18
SemyonSinchenko merged commit b047b77 into main May 1, 2026
20 checks passed
SemyonSinchenko deleted the alert-autofix-1 branch May 1, 2026 08:03
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL