-
-
Notifications
You must be signed in to change notification settings - Fork 629
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 recipe for php-bug-catcher/bug-catcher #1659
Conversation
tito10047
commented
Aug 19, 2024
Q | A |
---|---|
License | MIT |
Packagist | https://packagist.org/packages/php-bug-catcher/bug-catcher |
Thanks for the PR 😍 How to test these changes in your application
Diff between recipe versionsIn order to help with the review stage, I'm in charge of computing the diff between the various versions of patched recipes. |
Head branch was pushed to by a user without write access
Head branch was pushed to by a user without write access
Head branch was pushed to by a user without write access
My bundle is not only bundle, but whole application. and is target only for php 8.3 and symfony 7.1. so I can't pass your tests |
I can't get work this recipie. when ai run command |
A recipe should only be about the package you configure. Here, you have too many other configs. That cannot be merged as is. |
Yes, but this is not a package, this is the entire application. An application that you can easily expand with your own code. If you can't merge it, close this PR. But it's annoying to force people to set all this settings manually when they want to run my application. just see how huge is docs for this first run docs Do you have any recommendations on how to do it better? |
You can create and provide a setup command, creating such config? |
Ok, I will try, thanks :) |
Pull request was closed
This is good idea, but can't be work. When I install my budnle, then project is broken because missing configuration required for this bundle. So I can't run command which need to generate this config. Any other idea? |