Skip to content

fix: change permissions to read and update verification for cmake-set…#4963

Open
hanniavalera wants to merge 1 commit into
mainfrom
fix/generate-docs-fork-pr-verify
Open

fix: change permissions to read and update verification for cmake-set…#4963
hanniavalera wants to merge 1 commit into
mainfrom
fix/generate-docs-fork-pr-verify

Conversation

@hanniavalera
Copy link
Copy Markdown
Contributor

This pull request updates the documentation generation workflow to improve security and ensure that documentation is always up to date before merging. The main changes involve modifying permissions and replacing the automatic commit step with a verification step.

Workflow security and process improvements:

  • Changed the workflow's contents permission from write to read to reduce the risk of unintended repository modifications.
  • Removed the automatic commit and push step for updated documentation, replacing it with a verification step that checks if docs/cmake-settings.md is up to date. If the file is outdated, the workflow fails with an error message instructing the user to update and commit the file manually.

@hanniavalera hanniavalera requested a review from snehara99 June 8, 2026 16:02
@hanniavalera hanniavalera enabled auto-merge (squash) June 8, 2026 16:19
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