search

kylekatarnls / business-time

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

Fix add/sub time with local mode #32

Closed kylekatarnls closed 4 years ago

kylekatarnls commented 4 years ago

Fix #31

codecov[bot] commented 4 years ago

Codecov Report

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

Impacted file tree graph

@@             Coverage Diff             @@
##              master       #32   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
- Complexity       230       237    +7     
===========================================
  Files             16        16           
  Lines            517       528   +11     
===========================================
+ Hits             517       528   +11
Impacted Files Coverage Δ Complexity Δ
src/BusinessTime/Calculator.php 100.00% <100.00%> (ø) 24.00 <2.00> (+3.00)
src/BusinessTime/MixinBase.php 100.00% <100.00%> (ø) 48.00 <0.00> (+4.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 1909680...784c1bc. Read the comment docs.

codeclimate[bot] commented 4 years ago

Code Climate has analyzed commit 784c1bcd and detected 4 issues on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 2
Style 2

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.