Skip to content

ci: bump external PR notifications workflow to v8 - #1942

Merged
AlvaroBrey merged 1 commit into
mainfrom
ci/external-pr-notifications-v8
Sep 3, 2026
Merged

ci: bump external PR notifications workflow to v8#1942
AlvaroBrey merged 1 commit into
mainfrom
ci/external-pr-notifications-v8

Conversation

@AlvaroBrey

@AlvaroBrey AlvaroBrey commented Sep 3, 2026

Copy link
Copy Markdown
Member

Bumps the shared workflow to v8, which gates on head.repo.fork instead of author_association so PRs from org members with private membership stop being posted as external. See RevenueCat/sdk-github-workflows#11.


Note

Low Risk
CI-only workflow version bump; affects Slack notification targeting for opened PRs, not application runtime or secrets handling beyond the existing webhook.

Overview
Updates the External PR Notifications reusable workflow pin from v7 to v8 of revenuecat/sdk-github-workflows.

That shared release changes how “external” PRs are detected: notifications now key off head.repo.fork instead of author_association, so PRs opened by org members with private membership are no longer misclassified as external and posted to Slack.

Reviewed by Cursor Bugbot for commit f81a6fc. Bugbot is set up for automated code reviews on this repo. Configure here.

@AlvaroBrey
AlvaroBrey requested a review from a team as a code owner September 3, 2026 16:11
@AlvaroBrey AlvaroBrey added the pr:other A code change that improves performance label Sep 3, 2026
@AlvaroBrey AlvaroBrey self-assigned this Sep 3, 2026
@AlvaroBrey
AlvaroBrey requested a review from a team September 3, 2026 16:11
@AlvaroBrey
AlvaroBrey enabled auto-merge (squash) September 3, 2026 16:12
@AlvaroBrey
AlvaroBrey merged commit e14660f into main Sep 3, 2026
13 checks passed
@AlvaroBrey
AlvaroBrey deleted the ci/external-pr-notifications-v8 branch September 3, 2026 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:other A code change that improves performance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants