Alex Rattray

Results 411 comments of Alex Rattray

Thanks, we'll have this addressed shortly.

Sorry for the continued delay; this will land after https://github.com/openai/openai-openapi/pull/117 or equivalent is merged.

Thanks! This will be addressed via https://github.com/openai/openai-openapi/pull/109

Thanks for filing, I would like to do this at some point. Note that this applies to all forms of streams, not just web ReadableStreams (and the GCP libraries return...

Thank you. This relates to https://github.com/openai/openai-node/issues/271 but I think we would approach it differently. In this case, it seems it may be simplest to allow `params: FormData` in place of...

One thing to note is that with undici, it looks like `response.body` isn't a Node `Readable` stream, which would be breaking change for `node-fetch` users… any thoughts there?

> This is to spec so that code is actually interoperable. You mean, interoperable with web streams, not Node ones, right? So it will be breaking? Is there any way...

I'm probably leaning towards a major version bump in that case. I imagine shimming a web ReadableStream to also be compatible with a node Readable stream would be pretty gross....

Hey, sorry, just checking in – are you blocked on me here?

Sadly we don't run CI in public for this repo yet, just internally. The steps to test it locally are: ```sh yarn OPENAI_API_KEY=… yarn ts-node ecosystem-tests/cli.ts npx prism mock https://github.com/openai/openai-openapi/raw/master/openapi.yaml...