search

gpressutto5 / laravel-slack

:hash: Slack notification for Laravel as it should be. Easy, fast, simple and highly testable.
MIT License
289 stars 41 forks source link

Laravel 9.x Compatibility #23

Closed laravel-shift closed 2 years ago

laravel-shift commented 2 years ago

This is an automated pull request from Shift to update your package code and dependencies to be compatible with Laravel 9.x.

Before merging, you need to:

If you do find an issue, please report it by commenting on this PR to help improve future automation.

laravel-shift commented 2 years ago

:alembic: Using this package? If you would like to help test these changes or believe them to be compatible, you may update your project to reference this branch.

To do so, temporarily add Shift's fork to the repositories property of your composer.json:

{
    "repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/laravel-shift/laravel-slack.git"
        }
    ]
}

Then update your dependency constraint to reference this branch:

{
    "require": {
        "gpressutto5/laravel-slack": "dev-l9-compatibility",
    }
}

Finally, run: composer update

codecov[bot] commented 2 years ago

Codecov Report

Merging #23 (1000609) into master (dbe6f87) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##              master       #23   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
  Complexity        29        29           
===========================================
  Files              5         5           
  Lines             74        74           
===========================================
  Hits              74        74

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 dbe6f87...1000609. Read the comment docs.

azimidev commented 2 years ago

when this will be merged and released?

iambryansanders commented 2 years ago

Can we merge this?

zakariaarrid commented 2 years ago

Can we merge this please ? thanks.

lucaszrz commented 2 years ago

+1 would like this merge, pls ;)

gpressutto5 commented 2 years ago

I'm sorry folks, I didn't see this before.