-
Notifications
You must be signed in to change notification settings - Fork 884
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
release: 4.16.0 #430
release: 4.16.0 #430
Conversation
build-deno
Outdated
cat << EOF > deno/README.md | ||
# OpenAI Node API Library - Deno build | ||
|
||
This is a build produced from openai/openai-node - please go there to read the source, file issues, etc. | ||
This is a build produced from https://github.com/openai/openai-node - please go there to read the source and docs, file issues, etc. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
em-dash instead of hyphen?
build-deno
Outdated
\`\`\` | ||
import OpenAI from "https://deno.land/x/openai"; | ||
\`\`\`ts | ||
import OpenAI from "https://deno.land/x/openai@$PACKAGE_VERSION/mod.ts"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you also update the README of openai-node? That has Deno examples, and we should update them to use mod.ts
+ point to this repo.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah right of course, ty
ce13fc6
to
5da63d4
Compare
e5a5cdf
to
74bf336
Compare
…ll-e-3, tts, and more (#433)
3c5dae8
to
fb92f5e
Compare
🤖 Release is at https://github.com/openai/openai-node/releases/tag/v4.16.0 🌻 |
Thank you! 🙏 |
@@ -205,19 +359,24 @@ export interface ChatCompletionMessage { | |||
/** | |||
* The role of the author of this message. | |||
*/ | |||
role: ChatCompletionRole; | |||
role: 'assistant'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure if this will be noticed, but I'm curious why this was changed to just 'assistant' from ChatCompletionRole? The change broke our TS build that was relying on ChatCompletionMessage to define some messages before senging into openai.chat.completions.create() but maybe we shouldn't be doing that?
I'll review changing to ChatCompletionMessageParam as the type to use before we call openai.chat.completions.create
Automated Release PR
4.16.0 (2023-11-06)
Full Changelog: v4.15.4...v4.16.0
Features
Bug Fixes
Documentation
This Pull Request has been generated automatically as part of Stainless's release process.
For a better experience, it is recommended to use either rebase-merge or squash-merge when merging this pull request (see details).
More technical details can be found at stainless-api/release-please.