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

When using Laravel set default preset to laravel #9

Closed

Conversation

patrickbrouwers
Copy link

It might be logical to most people if using PhpInsights via Laravel that the preset is Laravel. It confused me at first when running php artisan insights after publishing the config; I was expecting it to "just" work :)

It might be logical to most people if using PhpInsights via Laravel that the preset is Laravel. It confused me at first when running php artisan insights after publishing the config; I was expecting it to "just" work :)
@nunomaduro
Copy link
Owner

@patrickbrouwers That's the default preset that will be used in other frameworks. Magento, Wordpress, etc.

You may need to create new stub inside the Laravel adapter folder.

@patrickbrouwers
Copy link
Author

Aha, I thought it was only used for Laravel. I'll wip up a new PR for it tomorrow.

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

Successfully merging this pull request may close these issues.

2 participants