Skip to content

fix(ci): add id-token permission for pr-notify workflow#72

Merged
alexs-mparticle merged 1 commit intomainfrom
fix/nojira-pr-notify
Mar 23, 2026
Merged

fix(ci): add id-token permission for pr-notify workflow#72
alexs-mparticle merged 1 commit intomainfrom
fix/nojira-pr-notify

Conversation

@alexs-mparticle
Copy link
Collaborator

Summary

  • The reusable oss_pr_opened_notification.yml workflow from ROKT/rokt-workflows requests id-token: write, but the calling workflow didn't grant it, defaulting to id-token: none
  • Adds permissions: id-token: write to the pr-notify job to match what the reusable workflow expects

Test plan

  • Verify the Reusable Workflows check no longer fails with the id-token permission error on new PRs

The reusable oss_pr_opened_notification workflow requests id-token: write,
but the calling workflow did not grant it, causing the workflow to fail.
@alexs-mparticle alexs-mparticle merged commit fa278ff into main Mar 23, 2026
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.

4 participants