Skip to content

fix(ci): add contents read permission for pr-notify workflow#73

Merged
alexs-mparticle merged 3 commits intomainfrom
fix/nojira-pr-notify
Mar 23, 2026
Merged

fix(ci): add contents read permission for pr-notify workflow#73
alexs-mparticle merged 3 commits intomainfrom
fix/nojira-pr-notify

Conversation

@alexs-mparticle
Copy link
Collaborator

Summary

  • Adds contents: read permission to the pr-notify job in the reusable workflows
  • The reusable workflow from ROKT/rokt-workflows needs this permission to check out the repo

Test plan

  • Verify the pr-notify job runs successfully on this PR

The reusable oss_pr_opened_notification workflow requests id-token: write,
but the calling workflow did not grant it, causing the workflow to fail.
The reusable workflow needs contents:read to check out the repo.
@alexs-mparticle alexs-mparticle merged commit 7c84013 into main Mar 23, 2026
3 of 5 checks passed
github-actions bot pushed a commit that referenced this pull request Mar 24, 2026
# [1.18.0](v1.17.0...v1.18.0) (2026-03-24)

### Bug Fixes

* **ci:** add contents read permission for pr-notify workflow ([#73](#73)) ([7c84013](7c84013))
* **ci:** add id-token permission for pr-notify workflow ([#72](#72)) ([fa278ff](fa278ff))

### Features

* pass mParticle session ID to Rokt SDK on launcher creation ([#74](#74)) ([1a5b591](1a5b591))
* sync Rokt session ID to mParticle via setIntegrationAttribute ([#67](#67)) ([021b2a5](021b2a5))
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