search

appstract / laravel-opcache

Laravel Package for OPcache
MIT License
1.34k stars 119 forks source link

Lumen route support. #22

Closed HyanCat closed 7 years ago

HyanCat commented 7 years ago

We know, the Route facade does not work in Lumen. I made a simple work that let the package work well for Lumen.

ovanschie commented 7 years ago

Thanks for your contribution :) I think a few thinks can be done simpler, also where is the different serviceProvider for? And how did you get Illuminate/Routing to get to work?

HyanCat commented 7 years ago

Yeah, you are right and simpler. But another problem is: the unused AuthorizesRequests and ValidatesRequests are not supported in Lumen too. I will close this PR and hope you can remove them in controller later. And, thanks for your nice work.