Skip to content

fix(ci): add contents: read to the credo job - #2817

Closed
joshprice wants to merge 2 commits into
mainfrom
fix/credo-job-contents-permission
Closed

fix(ci): add contents: read to the credo job#2817
joshprice wants to merge 2 commits into
mainfrom
fix/credo-job-contents-permission

Conversation

@joshprice

@joshprice joshprice commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

The credo job declares its own permissions:

  credo:
    permissions:
      security-events: write

A job-level permissions: block replaces the workflow default rather than extending it, so the job runs with no contents permission. actions/checkout then fails:

fatal: repository 'https://github.com/<org>/<repo>/' not found
The process '/usr/bin/git' failed with exit code 128

Checkout of a public repository succeeds without contents, so this only affects private repos calling ash-ci.yml. mix credo --strict passes on team-alembic/ash_authentication (public) and fails at checkout on a private repo on the same workflow.

credo is the only ungated job that overrides permissions. deploy-docs also overrides, but is gated on publish-docs and a repository-owner check.

@joshprice joshprice changed the title fix(ci): give the credo job contents: read so it can check out private repos fix(ci): add contents: read to the credo job Aug 2, 2026
@joshprice joshprice closed this Aug 2, 2026
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