DelSkayn / rquickjs

High level bindings to the quickjs javascript engine
MIT License
431 stars 58 forks source link

Fix some issues in the schedular #302

Closed DelSkayn closed 2 months ago

DelSkayn commented 2 months ago

This PR fixes some problems with the schedular.

codecov-commenter commented 2 months ago

Codecov Report

Attention: Patch coverage is 86.06061% with 46 lines in your changes are missing coverage. Please review.

Project coverage is 68.47%. Comparing base (84d50f3) to head (56041ef).

Files Patch % Lines
core/src/runtime/async.rs 68.90% 37 Missing :warning:
core/src/util.rs 80.64% 6 Missing :warning:
core/src/context/async/future.rs 94.44% 1 Missing :warning:
core/src/runtime/schedular.rs 98.33% 1 Missing :warning:
core/src/runtime/schedular/waker.rs 94.73% 1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #302 +/- ## ========================================== + Coverage 68.06% 68.47% +0.41% ========================================== Files 81 83 +2 Lines 12214 12375 +161 ========================================== + Hits 8313 8474 +161 Misses 3901 3901 ```

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