Skip to content

ci: grant pull-requests permission to docs workflow call - #187

Merged
ChrisRackauckas merged 1 commit into
SciML:mainfrom
ChrisRackauckas-Claude:docs/workflow-permissions
Sep 17, 2026
Merged

ChrisRackauckas merged 1 commit into
SciML:mainfrom
ChrisRackauckas-Claude:docs/workflow-permissions

Conversation

@ChrisRackauckas-Claude

Copy link
Copy Markdown
Member

Please ignore until reviewed by @ChrisRackauckas.

What changed and why

Docs runs on main die with empty startup_failure (e.g. https://github.com/SciML/ADTypes.jl/actions/runs/34655414874). The reusable documentation workflow's build job requests actions, contents, pull-requests, and statuses write permissions, and a caller must grant at least that set or the run fails closed at startup. This file granted everything except pull-requests. One-line fix.

Same root cause as SciML/DataCollocations.jl#54 (where the caller granted nothing and needed the full block): the reusable workflow started requesting these permissions on Sep 9, matching exactly when these repos' docs runs began failing at startup.

Verification

  • A green run of this PR's Documentation workflow proves the fix (pending; will update).
  • Content itself is healthy: last completed docs build (Sep 9) succeeded.

Not verified

  • Deployed rendering (follows automatically once main docs complete).

🤖 Generated with OpenCode (model: muse-spark-1.3-contributor-free); no public session URL (local session at /home/crackauc/sandbox/tmp_20260917_040834_91074).

The reusable documentation workflow requests actions, contents,
pull-requests, and statuses write permissions. Callers must grant at
least that set or the run fails closed at startup; this file was
missing pull-requests.

Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Agent-Harness: OpenCode
Agent-Model: muse-spark-1.3-contributor-free
Agent-Session: local session at /home/crackauc/sandbox/tmp_20260917_040834_91074 (no public conversation URL)
@ChrisRackauckas
ChrisRackauckas marked this pull request as ready for review September 17, 2026 13:31
@ChrisRackauckas
ChrisRackauckas merged commit e242081 into SciML:main Sep 17, 2026
14 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