-
Notifications
You must be signed in to change notification settings - Fork 127
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
Suggestion for modules' web pages: Add a hint to all methods of a certain trait #529
Comments
the laravel module also splits its assertions into Traits, and the Lumen module will do the same soon. I think it would be useful to be able to docblocks for traits. Or even more, to be able to somehow organize the assertions in the website documentation according to the trait to which they belong. Keep things that are related together, right? |
I already suggested one step into this direction at Codeception/Codeception#6063 (comment):
But after thinking some more about it, your idea is even better! Let's drop the alphabetical order in favor of trait "sections" - both in the page and in the index column (right hand side) Why?
So the solution would then be to have a docblock for the trait, and this is displayed as "introduction" of this trait's section on the page. So (for now) repeating stuff at multiple functions isn't even necessary. |
This would also make my idea to add some "see also" links obsolete, which I started at Codeception/module-symfony#122 |
It would be nice to have way to somehow add a hint to some trait (e.g. https://github.com/Codeception/module-symfony/blob/main/src/Codeception/Module/Symfony/SecurityAssertionsTrait.php), and then have this hint displayed at all methods of that trait (e.g. https://codeception.com/docs/modules/Symfony#dontSeeAuthentication).
Background: That security trait requires Symfony's
security.helper
service to be declaredpublic
, see Codeception/module-symfony#138, and there might be more use cases in the future where something has to be explained for an entire trait.@Naktibalda what do you think?
The text was updated successfully, but these errors were encountered: