PrefectHQ / marvin

✨ Build AI interfaces that spark joy
https://askmarvin.ai
Apache License 2.0
5.34k stars 348 forks source link

Only support async handlers #885

Closed jlowin closed 8 months ago

jlowin commented 8 months ago

The split in the openAI API makes it difficult to support both async and sync event handlers at this time without a number of forking paths. It's simpler to just support async event handlers.

Note: this is technically a bugfix because today we document that we support sync handlers but they error if you try to use them.