zazuko / kopflos

kopflos - Linked Data APIs
MIT License
14 stars 6 forks source link

fix: unhandled errors in promise would crash express #167

Closed tpluscode closed 2 months ago

changeset-bot[bot] commented 2 months ago

🦋 Changeset detected

Latest commit: 8963b509b594521c5142dc83af0a3bf0f615ab7c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package | Name | Type | | ----------------- | ----- | | @kopflos-cms/core | Patch |

Not sure what this means? Click here to learn what changesets are.

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

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 95.47170% with 12 lines in your changes missing coverage. Please review.

Project coverage is 97.25%. Comparing base (619d7f2) to head (8963b50). Report is 20 commits behind head on master.

Files with missing lines Patch % Lines
packages/core/lib/Kopflos.ts 90.99% 8 Missing and 2 partials :warning:
packages/express/BodyWrapper.ts 94.73% 2 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #167 +/- ## ========================================== - Coverage 97.94% 97.25% -0.70% ========================================== Files 17 19 +2 Lines 584 764 +180 Branches 55 79 +24 ========================================== + Hits 572 743 +171 - Misses 12 19 +7 - Partials 0 2 +2 ```

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

tpluscode commented 2 months ago

Ok, now I incorporated #161 in this PR