search

kylekatarnls / business-time

Carbon mixin to handle business days and opening hours
MIT License
297 stars 14 forks source link

Implement #25 diff methods #27

Closed kylekatarnls closed 4 years ago

kylekatarnls commented 4 years ago

Closes #25

codecov[bot] commented 4 years ago

Codecov Report

Merging #27 into master will not change coverage. The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##              master       #27   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
- Complexity       203       228   +25     
===========================================
  Files             14        16    +2     
  Lines            453       508   +55     
===========================================
+ Hits             453       508   +55
Impacted Files Coverage Δ Complexity Δ
src/BusinessTime/MixinBase.php 100.00% <ø> (ø) 44.00 <0.00> (ø)
src/BusinessTime/DiffCalculator.php 100.00% <100.00%> (ø) 15.00 <15.00> (?)
src/BusinessTime/Traits/Diff.php 100.00% <100.00%> (ø) 10.00 <10.00> (?)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 1f6260a...84c9e27. Read the comment docs.

codeclimate[bot] commented 4 years ago

Code Climate has analyzed commit 84c9e276 and detected 32 issues on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 3
Style 27
Clarity 2

Note: there is 1 critical issue.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 100.0%.

View more on Code Climate.