denoland / deno_std

The Deno Standard Library
https://jsr.io/@std
MIT License
2.83k stars 582 forks source link

refactor(yaml): cleanup `mergeMappings()` #5314

Closed timreichen closed 2 days ago

timreichen commented 2 days ago

Thank you.

Turns out Object.defineProperty is used because sometimes a prototype is passed...

codecov[bot] commented 2 days ago

Codecov Report

Attention: Patch coverage is 88.88889% with 1 line in your changes missing coverage. Please review.

Project coverage is 95.44%. Comparing base (f0057a8) to head (61b1b4c).

Files Patch % Lines
yaml/_loader.ts 88.88% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #5314 +/- ## ========================================== - Coverage 95.81% 95.44% -0.37% ========================================== Files 458 458 Lines 37877 37876 -1 Branches 5563 5533 -30 ========================================== - Hits 36292 36151 -141 - Misses 1545 1685 +140 Partials 40 40 ```

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