inngest / inngest-js

The developer platform for easily building reliable workflows with zero infrastructure for TypeScript & JavaScript
https://www.inngest.com/
GNU General Public License v3.0
414 stars 41 forks source link

Loosen typing on `match` to remove performance concerns #644

Closed jpwilliams closed 2 months ago

jpwilliams commented 2 months ago

Summary

Loosens typing of match in step.waitForEvent() and cancelOn configurations in order to remove TypeScript performance concerns when many complex events are used. It is now simply any string and is unvalidated against event typing.

This supersedes #642 and marks match fields as deprecated; they will be removed fully or reworked in v4 as we coalesce towards a solid solution for writing matching expressions for these tools.

Checklist

Related

changeset-bot[bot] commented 2 months ago

🦋 Changeset detected

Latest commit: 81ec7589b6a42c8d22c472ec69dfcae78f32adaf

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package | Name | Type | | ------- | ----- | | inngest | Patch |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR