search

kylekatarnls / business-time

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

Fix #39 Keep local hours for diff calculation #40

Closed kylekatarnls closed 3 years ago

kylekatarnls commented 3 years ago

Fix #39

codeclimate[bot] commented 3 years ago

Code Climate has analyzed commit 9d228de8 and detected 0 issues on this pull request.

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% (0.0% change).

View more on Code Climate.

codecov[bot] commented 3 years ago

Codecov Report

Merging #40 (9d228de) into master (73c61f8) will not change coverage. The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##              master       #40   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
- Complexity       245       253    +8     
===========================================
  Files             16        16           
  Lines            551       562   +11     
===========================================
+ Hits             551       562   +11
Impacted Files Coverage Δ Complexity Δ
src/BusinessTime/MixinBase.php 100.00% <ø> (ø) 49.00 <0.00> (+1.00)
src/BusinessTime/DiffCalculator.php 100.00% <100.00%> (ø) 23.00 <7.00> (+7.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 73c61f8...9d228de. Read the comment docs.