search

spatie / laravel-varnish

Making Varnish and Laravel play nice together
https://freek.dev/663-using-varnish-on-a-laravel-forge-provisioned-server
MIT License
406 stars 44 forks source link

Complete the requested changes of #37 #50

Closed Daniyal-Javani closed 3 years ago

freekmurze commented 3 years ago

Constructors of all commands are run on each request. It might be better to remove it, and to have $this-varnish = app(Varnish::class) in the handle class of the command.

Daniyal-Javani commented 3 years ago

Yeah, I've changed the commit.

spatie-bot commented 3 years ago

Dear contributor,

because this pull request seems to be inactive for quite some time now, I've automatically closed it. If you feel this pull request deserves some attention from my human colleagues feel free to reopen it.