deepkit / deepkit-framework

A new full-featured and high-performance TypeScript framework
https://deepkit.io/
MIT License
3.24k stars 123 forks source link

feat(injector): improve set method api #557

Closed marcus-sa closed 8 months ago

marcus-sa commented 8 months ago

Summary of changes

Relinquishment of Rights

Please mark following checkbox to confirm that you relinquish all rights of your changes:

codecov-commenter commented 8 months ago

Codecov Report

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

Project coverage is 78.02%. Comparing base (3e09ddf) to head (0221a8b). Report is 43 commits behind head on master.

:exclamation: Current head 0221a8b differs from pull request most recent head 901bd24. Consider uploading reports for the commit 901bd24 to get more accurate results

Files Patch % Lines
packages/rpc/src/decorators.ts 75.00% 4 Missing :warning:
packages/rpc/src/server/action.ts 93.33% 4 Missing :warning:
packages/rpc/src/server/kernel.ts 92.45% 4 Missing :warning:
packages/type/src/serializer.ts 75.00% 2 Missing :warning:
packages/type/src/utils.ts 0.00% 2 Missing :warning:
packages/broker/src/kernel.ts 75.00% 1 Missing :warning:
packages/type/src/reflection/reflection.ts 80.00% 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 #557 +/- ## ========================================== - Coverage 78.06% 78.02% -0.05% ========================================== Files 191 149 -42 Lines 20296 18291 -2005 Branches 5255 4864 -391 ========================================== - Hits 15844 14271 -1573 + Misses 4452 4020 -432 ```

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

marcj commented 8 months ago

great, thanks!