Skip to content

Comments

feat (docs): Helicone observability#5472

Merged
lgrammel merged 6 commits intovercel:mainfrom
colegottdank:helicone-vercel
Apr 4, 2025
Merged

feat (docs): Helicone observability#5472
lgrammel merged 6 commits intovercel:mainfrom
colegottdank:helicone-vercel

Conversation

@colegottdank
Copy link
Contributor

No description provided.

@lgrammel
Copy link
Collaborator

lgrammel commented Apr 2, 2025

  • Please refer to the AI SDK as AI SDK (not Vercel AI)
  • Please follow the structure of other observability providers, in particular start with a Setup section and try to align section names
  • Please reduce marketing language to a reasonable degree - this is documentation. E.g. Setting up Helicone takes just three simple steps: -> You can set up Helicone as follows or Setting up Helicone:

@colegottdank colegottdank reopened this Apr 2, 2025
@colegottdank
Copy link
Contributor Author

  • Please refer to the AI SDK as AI SDK (not Vercel AI)
  • Please follow the structure of other observability providers, in particular start with a Setup section and try to align section names
  • Please reduce marketing language to a reasonable degree - this is documentation. E.g. Setting up Helicone takes just three simple steps: -> You can set up Helicone as follows or Setting up Helicone:

I pushed out fixes, please let me know.
Regarding the headers, I adjusted the Setup section and attempted to align the other section names, however I noticed they were quite different between providers, so I went with best effort.

@lgrammel
Copy link
Collaborator

lgrammel commented Apr 3, 2025

Please run pnpm prettier-fix

@colegottdank
Copy link
Contributor Author

Please run pnpm prettier-fix

Done!

@lgrammel lgrammel merged commit bc46777 into vercel:main Apr 4, 2025
6 of 7 checks passed
lgrammel pushed a commit that referenced this pull request Apr 4, 2025
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