-
Notifications
You must be signed in to change notification settings - Fork 7
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 a note if a suggested package is needed #7
Labels
Comments
Related to #6 |
Closed
17 tasks
I'm absolutely for this. I've been caught out in the past because packages were marked as optional in Composer, but were mandatory at runtime. |
I think we can test if the content is available with the install command via script. |
Agreed. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We should add a note to the documentation if a class needs another component which is not installed by Composer because it is not strictly required.
zend-validator
https://github.com/zendframework/zend-validator/blob/87e743d11d49a157011a49a962c25e7a11adc7c5/composer.json#L34-L43
zend-paginator
https://github.com/zendframework/zend-paginator/blob/b4e853ef104101069e781f50f662f514e4e4e33f/composer.json#L36-L43
zend-mail
https://github.com/zendframework/zend-mail/blob/0e43bfe8994f38b2c7a2c560b1dcfee6eaaabc37/composer.json#L33-L36
Example
An example can be found in documentation of zend-hydrator:
(Should be formatted as a blockquote.)
https://docs.zendframework.com/zend-hydrator/aggregate/#installation-requirements
The text was updated successfully, but these errors were encountered: