search

kylekatarnls / business-time

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

Skip instanciation if current date class match #50

Closed kylekatarnls closed 3 years ago

kylekatarnls commented 3 years ago

Fix #49

codecov[bot] commented 3 years ago

Codecov Report

Merging #50 (db4b57e) into master (f77379a) will not change coverage. The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##              master       #50   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
- Complexity       223       225    +2     
===========================================
  Files             16        16           
  Lines            570       573    +3     
===========================================
+ Hits             570       573    +3
Impacted Files Coverage Δ
src/BusinessTime/MixinBase.php 100.00% <100.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 f77379a...db4b57e. Read the comment docs.

codeclimate[bot] commented 3 years ago

Code Climate has analyzed commit db4b57e6 and detected 2 issues on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 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.