-
Notifications
You must be signed in to change notification settings - Fork 9
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
Function Hints #42
Comments
I've always preferred it in the docblock for script. Can you clarify what you mean by the engines preferring it via an attribute? I just tested on TryCF and dumping the function directly: ACF displays the docblock text as the hint back to at least CF10 and Lucee 5 shows the hint under the Test: https://trycf.com/gist/fcf6d216f21c10c7d7976897592f6860/acf?theme=monokai |
@KamasamaK ah, I made a silly mistake when testing this, I was using this:
Which wasn't showing the hint. If you use Sounds like this would have to be option, as I also prefer it in the comment block. |
@pfreitag That makes sense now. I was about to say in my other post that a docblock is not just a comment, since that's the word you used, but I thought you might have just misspoke. |
Are there any plans to add the |
Currently the function hint attribute is moved to a comment above the function. Both Lucee and ACF prefer that it is in a function argument instead (Which show up in the meta data when you dump the function).
Should become:
The text was updated successfully, but these errors were encountered: