Skip to content

fix(provider/gateway): remove duplicate PR_COUNT assignment + update owner of model settings workflow#13982

Merged
sylviezhang37 merged 9 commits intomainfrom
fix-slack-notif-config-error
Apr 2, 2026
Merged

fix(provider/gateway): remove duplicate PR_COUNT assignment + update owner of model settings workflow#13982
sylviezhang37 merged 9 commits intomainfrom
fix-slack-notif-config-error

Conversation

@sylviezhang37
Copy link
Copy Markdown
Contributor

@sylviezhang37 sylviezhang37 commented Mar 31, 2026

Background

Bot's commit left a duplicate PR_COUNT= assignment, causing a shell syntax error in the notify job. Sorry I should've reviewed it more closely!

Also updating owner ahead of my last day at Vercel.

Summary

  • Removed duplicate line
  • Restored GITHUB_OUTPUT write for Slack step
  • Updated owner to Rohan T.

Testing

Tested with my test slack channel

Checklist

  • Tests have been added / updated (for bug fixes / features)
  • Documentation has been added / updated (for bug fixes / features)
  • A patch changeset for relevant packages has been added (for bug fixes / features - run pnpm changeset in the project root)
  • I have reviewed this pull request (self-review)

@tigent tigent bot added the maintenance CI, internal documentation, automations, etc label Mar 31, 2026
@sylviezhang37 sylviezhang37 changed the title fix(provider/gateway): remove duplicate PR_COUNT assignment fix(provider/gateway): remove duplicate PR_COUNT assignment + update owner Apr 2, 2026
@sylviezhang37 sylviezhang37 changed the title fix(provider/gateway): remove duplicate PR_COUNT assignment + update owner fix(provider/gateway): remove duplicate PR_COUNT assignment + update owner of model settings workflow Apr 2, 2026
@sylviezhang37 sylviezhang37 merged commit a060501 into main Apr 2, 2026
17 checks passed
@sylviezhang37 sylviezhang37 deleted the fix-slack-notif-config-error branch April 2, 2026 18:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance CI, internal documentation, automations, etc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants