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
type-graph.txt: what is it good for? #2584
Comments
Whilst I find the Type Graphs in the documentation to be pretty, I have never found them useful. I could not work out what to do with them. However, I could see a use for them. If I could trace in which class some function come, eg. is Also I have found that some graphs are so small as to be almost meaningless, whilst others are so large that it is difficult to read everything. My suggestion is to eliminate the TypeGraphs from the documentation system until a better way of achieving the aim of increasing clarity has been demonstrated. |
While including graphs or not would be a worthwhile debate, let's try and
stick to the point, which is the fact that we are using an external file
for metadata that can actually found by introspecting the class itself.
|
I think one of the original reasons not to use introspection was that we might want to hide some details from the users, like for example If we switch to introspection, we need at least some form of customization for it. If we have that, I have no problem with getting rid of it. |
@moritz we could achieve that with on-line blacklists instead of an off-line whitelist; that is, customization as you say. |
This is included now in the document metadata proposal |
The problem
type-graph.txt
was created 7 years ago as aFrom them on, it's outdated both ways and I don't really see the need for it, since there's introspection both into the classes and into the roles (although there might be some problem with NQP roles, it's not unsurmountable, I think).
Suggestions
As part of the redesign of the documentation system #2542 (see also this), we should consider getting rid of it and populate Perl6::Type (#2573) by using the metamodel entirely.
The text was updated successfully, but these errors were encountered: