freekmurze commented 2 years ago

This can already be safely merged in.

Could you tag a new release after merging?

Thank you!! 🙌

codecov[bot] commented 2 years ago

Codecov Report

Merging #40 (4e182be) into main (0a14e87) will not change coverage. The diff coverage is n/a.

:exclamation: Current head 4e182be differs from pull request most recent head 473530d. Consider uploading reports for the commit 473530d to get more accurate results

@@             Coverage Diff             @@
##                main       #40   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
  Complexity        66        66           
===========================================
  Files              8         8           
  Lines            173       173           
===========================================
  Hits             173       173

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 0a14e87...473530d. Read the comment docs.

shengslogar commented 2 years ago

Thank you!! Appreciate all the work you do. 🥂

freekmurze commented 2 years ago

Tried to fix the failing tests, hopefully they will pass now.

shengslogar commented 2 years ago

Beat me to it. I'm actually going to replace with \Orchestra\Testbench\Foundation\Console\Kernel. Will push and merge shortly.

shengslogar commented 2 years ago

Tagged v7.1.1

diglactic / laravel-breadcrumbs

Add support for Laravel 9 #40

Codecov Report