search

graphile / crystal

🔮 Graphile's Crystal Monorepo; home to Grafast, PostGraphile, pg-introspection, pg-sql2 and much more!
https://graphile.org/
Other
12.58k stars 570 forks source link

OutputPlan rewritten to use TypeScript rather than eval'd JS #2118

Closed benjie closed 2 months ago

benjie commented 2 months ago

This removes our optimization but in doing so it makes the code easier to refactor. We'll need that for some upcoming changes. We may re-introduce the optimization in later releases, likely after v5.0.0 has gone live.

changeset-bot[bot] commented 2 months ago

⚠️ No Changeset found

Latest commit: d689729aa71f9b69c1f9d5087c547865cb89e3ff

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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