-
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
New outline for the component documentation #13
Comments
Can I be a bit annoying and say that testing, at least a basic introduction, should be a requirement. I've read through more than enough components where testing was an after-thought, which turned out to be clear why it was. Because when I attempted to test the component in my code, it was often downright impossible. I'm not suggesting that this be verbose, but at least a base minimum, which should also help to build the perception of professionalism. I'd see security and performance basics, perhaps, as optional extras though. |
The list from above contains already:
This means to add descriptions how components like laminas-mail, laminas-form, laminas-form and so can be used in unit tests. |
Thanks for the feedback. Seems good to me. |
For consistency and simplification of the component documentation I would suggest a new outline / structure for a component. (This proposed structure is not related to the structure of a single page!)
Goal
Suggestion
Some principles that should be met:
New structure of the outline:
Key Benefits
Implementation
The implementation can not be done in one single step. It must be done separately for each component.
Example (1)
The following list is a concrete example based on laminas-stratigility:
Essentials
Middleware
Migration
Example (2)
The following list is an example for laminas-hydrator:
Essentials
Hydrators
Strategies for Hydration and Extraction
Naming Strategies
Filters
Iterators
Application Integration
Cookbook
Example (3)
laminas-i18n already implements the new structure: https://docs.laminas.dev/laminas-i18n/
The text was updated successfully, but these errors were encountered: