di
di copied to clipboard
Lazy services
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 |
Tests fail because of https://github.com/yiisoft/factory/pull/116 isn't in master
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.
Adding benchmarking would be great.