di icon indicating copy to clipboard operation
di copied to clipboard

Lazy services

Open xepozz opened this issue 4 years ago • 3 comments
trafficstars

Q A
Is bugfix?
New feature? ✔️
Breaks BC?
Related issues https://github.com/yiisoft/di/issues/197, https://github.com/yiisoft/definitions/pull/58

xepozz avatar Aug 01 '21 10:08 xepozz

Tests fail because of https://github.com/yiisoft/factory/pull/116 isn't in master

xepozz avatar Aug 01 '21 10:08 xepozz

Codecov Report

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

Project coverage is 98.54%. Comparing base (aeb0856) to head (fedff5e).

:exclamation: Current head fedff5e differs from pull request most recent head 51fefa2. Consider uploading reports for the commit 51fefa2 to get more accurate results

Files Patch % Lines
src/Container.php 53.33% 7 Missing :warning:
Additional details and impacted files
@@              Coverage Diff              @@
##              master     #232      +/-   ##
=============================================
- Coverage     100.00%   98.54%   -1.46%     
- Complexity       161      168       +7     
=============================================
  Files             11       11              
  Lines            466      481      +15     
=============================================
+ Hits             466      474       +8     
- Misses             0        7       +7     

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

codecov[bot] avatar Nov 20 '22 08:11 codecov[bot]

Adding benchmarking would be great.

rustamwin avatar Dec 05 '22 10:12 rustamwin