Skip to content

feat: use contents read permission for CodeQL and semgrep analysis jobs.#182

Merged
LillieEntur merged 6 commits intomainfrom
feature/SIK-1886
Mar 18, 2026
Merged

feat: use contents read permission for CodeQL and semgrep analysis jobs.#182
LillieEntur merged 6 commits intomainfrom
feature/SIK-1886

Conversation

@LillieEntur
Copy link
Contributor

@LillieEntur LillieEntur commented Mar 16, 2026

💡 What does this PR do?

The pull request aims to remove usage of contents: write during workflow jobs that execute gradle jobs.

contents: write is only needed for uploading dependency graph, and only were required when the action was running in main branch, and no cached results were found. A new job were made to isolate the contents: write to only upload the dependency graph.

Closes: SIK-1886

🔧 List of changes

  • code-scan.yml
    • explicitly define github-token as input to github actions instead of being hidden by default input
    • replace contents: write with contents: read for semgrep-analysis and codeql-analysis jobs.
    • add job upload-dependency-graph

📋 Checklist

  • Am familiar with the release pipeline for this project
  • I have verified that the project runs as expected after the new changes

@LillieEntur LillieEntur marked this pull request as ready for review March 17, 2026 08:45
@LillieEntur LillieEntur requested a review from a team as a code owner March 17, 2026 08:45
@LillieEntur LillieEntur merged commit adeb7be into main Mar 18, 2026
4 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.

2 participants