Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for Lumen #92

Open
spiritix opened this issue May 19, 2020 · 3 comments
Open

Add support for Lumen #92

spiritix opened this issue May 19, 2020 · 3 comments

Comments

@spiritix
Copy link
Owner

The library has not yet been tested with Lumen, it seems that there are some issues at least.

@merlinblack
Copy link

We are using lada-cache in production on Lumen. As far as I can remember we only needed to provide a config_path function.

`<?php

if ( ! function_exists('config_path'))
{
/**
* Get the configuration path.
*
* @param string $path
* @return string
*/
function config_path($path = '')
{
return app()->basePath() . '/config' . ($path ? '/' . $path : $path);
}
}
`

@michael-rubel
Copy link

Looks like it's zero point to keep supporting Lumen now, due to the release of Laravel Octane. Lumen users should upgrade applications to Laravel if they wanna keep their apps up with new features and performance boosts.

@spiritix
Copy link
Owner Author

Yeah I agree, if the only reason for using Lumen was performance, then it's better to upgrade to Laravel anyway. Framework bootstrap only makes a very small part of application loading time, and nowadays with PHP8 and even Octane it's pretty much negligible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants