Skip to content

fix(node): Fix name of vercelAIIntegration to VercelAI #15298

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Feb 5, 2025

Conversation

mydea
Copy link
Member

@mydea mydea commented Feb 5, 2025

We use capitalized integration names everywhere, except here.

Noticed this by chance, this is kind of breaking, so probably also should go into the migration guide - @lforst should I add this, but this will conflict with the big PR, so maybe we just add it there directly?

We use capitalized integration names everywhere, except here.
@mydea mydea requested review from lforst and Lms24 February 5, 2025 09:36
@mydea mydea self-assigned this Feb 5, 2025
@lforst
Copy link
Member

lforst commented Feb 5, 2025

I'll add it!

Copy link

codecov bot commented Feb 5, 2025

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
205 1 204 8
View the full list of 1 ❄️ flaky tests
sdk.test.ts SDK-internal behaviorInjects fetch proxy script for SvelteKit<2.16.0

Flake rate in main: 40.00% (Passed 9 times, Failed 6 times)

Stack Traces | 0.154s run time
sdk.test.ts:4:3 Injects fetch proxy script for SvelteKit<2.16.0

To view more test analytics, go to the Test Analytics Dashboard
📢 Thoughts on this report? Let us know!

@mydea mydea merged commit 8506119 into develop Feb 5, 2025
100 of 102 checks passed
@mydea mydea deleted the fn/fix-vercel-ai-name branch February 5, 2025 10:20
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.

None yet

3 participants