-
Notifications
You must be signed in to change notification settings - Fork 826
Fold in comments from is_callable() page #689
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
Conversation
|
Maybe add a note on visibility of the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
|
Revised. |
|
@Girgias, what do you think about the layout of the notes section? It's rather different from what we usually use (i.e. multiple |
|
The layout might make more sense then what we currently have at times, so it's all green on my end. :-) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some post-merge comments. Poor effort from the code reviewers on this one. 😛
I'm not sure this is structured properly. I can restructure to some other format if someone tells me what the format should be. :-)