Skip to content

fix sourcify_all_chains permissions#1845

Merged
OmarTawfik merged 1 commit into
mainfrom
OmarTawfik/fix-sourcify-permissions
Jun 9, 2026
Merged

fix sourcify_all_chains permissions#1845
OmarTawfik merged 1 commit into
mainfrom
OmarTawfik/fix-sourcify-permissions

Conversation

@OmarTawfik

Copy link
Copy Markdown
Contributor

Child workflow succeeds on its own, but requires permissions to be passed explicitly when invoked from parent workflow.

The nested job 'singleShard' is requesting 'contents: read', but is only allowed 'contents: none'.

Child workflow succeeds on its own, but requires permissions to be passed explicitly when invoked from parent workflow.

[Failed build example](https://github.com/NomicFoundation/slang/actions/runs/27197438935):

> The nested job 'singleShard' is requesting 'contents: read', but is only allowed 'contents: none'.
@changeset-bot

changeset-bot Bot commented Jun 9, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 123f750

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

🐰 Bencher Report

BranchOmarTawfik/fix-sourcify-permissions
Testbedci

⚠️ WARNING: Truncated view!

The full continuous benchmarking report exceeds the maximum length allowed on this platform.

⚠️ WARNING: No Threshold found!

Without a Threshold, no Alerts will ever be generated.

🐰 View full continuous benchmarking report in Bencher

@OmarTawfik OmarTawfik marked this pull request as ready for review June 9, 2026 10:18
@OmarTawfik OmarTawfik requested review from a team as code owners June 9, 2026 10:18
@nebasuke

nebasuke commented Jun 9, 2026

Copy link
Copy Markdown
Member

Thanks for the fix!

@OmarTawfik OmarTawfik added this pull request to the merge queue Jun 9, 2026
Merged via the queue into main with commit 7feb5c1 Jun 9, 2026
123 of 153 checks passed
@OmarTawfik OmarTawfik deleted the OmarTawfik/fix-sourcify-permissions branch June 9, 2026 15:50
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