tailcallhq / tailcall

High Performance GraphQL Runtime
https://tailcall.run
Apache License 2.0
1.28k stars 244 forks source link

fix(jit): nested resolver handling #2338

Closed meskill closed 2 months ago

meskill commented 3 months ago

Summary:
Briefly describe the changes made in this PR.

Issue Reference(s):
Fixes #... (Replace "..." with the issue number)

Build & Testing:

Checklist:

codecov[bot] commented 3 months ago

Codecov Report

Attention: Patch coverage is 90.56604% with 10 lines in your changes missing coverage. Please review.

Project coverage is 86.02%. Comparing base (92455fd) to head (81271fb).

Files Patch % Lines
src/core/jit/store.rs 83.72% 7 Missing :warning:
src/core/jit/common/json_placeholder.rs 87.50% 2 Missing :warning:
src/core/jit/synth/synth_const.rs 96.55% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #2338 +/- ## ========================================== - Coverage 86.05% 86.02% -0.03% ========================================== Files 235 234 -1 Lines 22551 22407 -144 ========================================== - Hits 19406 19276 -130 + Misses 3145 3131 -14 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

tusharmath commented 2 months ago

Nice work @meskill!