denoland / deno_core

The core engine at the heart of Deno
MIT License
268 stars 87 forks source link

Rename op2 to op #808

Open littledivy opened 3 months ago

littledivy commented 3 months ago

It has stayed op2 for a long time now with older op macro removed

codecov-commenter commented 3 months ago

Codecov Report

Attention: Patch coverage is 70.90164% with 71 lines in your changes missing coverage. Please review.

Project coverage is 81.75%. Comparing base (0c7f83e) to head (732d2e4). Report is 50 commits behind head on main.

Files Patch % Lines
core/runtime/ops.rs 76.40% 21 Missing :warning:
core/ops_builtin_v8.rs 56.09% 18 Missing :warning:
core/ops_builtin.rs 37.03% 17 Missing :warning:
testing/checkin/runner/ops.rs 20.00% 8 Missing :warning:
ops/op/mod.rs 78.57% 3 Missing :warning:
testing/checkin/runner/ops_error.rs 66.66% 2 Missing :warning:
core/extensions.rs 0.00% 1 Missing :warning:
ops/lib.rs 80.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #808 +/- ## ========================================== + Coverage 81.43% 81.75% +0.32% ========================================== Files 97 98 +1 Lines 23877 24324 +447 ========================================== + Hits 19445 19887 +442 - Misses 4432 4437 +5 ```

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