Skip to content

ci: set workflow permissions#115

Merged
eoaksnes merged 1 commit intomainfrom
ci/update-workflow-permissions
Mar 5, 2026
Merged

ci: set workflow permissions#115
eoaksnes merged 1 commit intomainfrom
ci/update-workflow-permissions

Conversation

@eoaksnes
Copy link
Copy Markdown
Collaborator

@eoaksnes eoaksnes commented Mar 5, 2026

This pull request updates several GitHub Actions workflow files to explicitly set permissions for improved security and clarity. The changes ensure that each workflow only requests the minimum permissions needed for its tasks, following GitHub best practices.

Permissions configuration updates:

  • Added contents: read permission to the top-level of .github/workflows/on-push-main.yml, .github/workflows/on-push.yml, and .github/workflows/tests.yml to allow workflows to access repository contents. [1] [2] [3]
  • Added contents: write permission to the Build and deploy docs to Github pages job in .github/workflows/on-push-main.yml so it can publish documentation to GitHub Pages.
  • Added issues: write and pull-requests: write permissions to .github/workflows/stale.yml to allow the workflow to mark issues and pull requests as stale.

Simplification and cleanup:

  • Removed redundant and unnecessary permissions from the jobs: section of .github/workflows/codeql.yml to streamline the workflow configuration.

@eoaksnes eoaksnes requested a review from a team as a code owner March 5, 2026 12:06
@eoaksnes eoaksnes merged commit 4fc55bc into main Mar 5, 2026
4 checks passed
@eoaksnes eoaksnes deleted the ci/update-workflow-permissions branch March 5, 2026 12:06
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