-
Notifications
You must be signed in to change notification settings - Fork 2
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
propose: Allow execute functions #13
Comments
So you don't test these kind of functions? I don't really like breaking this kind of structure since commands are really easy to make via Especially since all that Command class needs to have is the |
Sorry, the example I gave was basic.
In this case, we have a test for CriticalRenewAuditLog class. For some security reasons (that I don't know), in my team, it is not allowed to create cake commands for certain things since they could be invoked by other parts of the development team, for example the 'system administrators', since they can log in the container/server/whatever and execute them by hand with ./bin/cake etc... |
Hello, in this commit: crhdev@83b5841 , I was playing a little to be able to allow the execution of functions instead of cake command. In my application, I have some use cases where creating a command is unnecessary. Do you think it may be possible to integrate something like this?
The text was updated successfully, but these errors were encountered: