-
Notifications
You must be signed in to change notification settings - Fork 292
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
Document Metamodel::TypePretense and Metamodel::MethodDelegation #3352
base: main
Are you sure you want to change the base?
Conversation
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.
Please check comments out.
doc/Type/Metamodel/CoercionHOW.pod6
Outdated
|
||
In signatures, I<type coercions> may be used to change the types of | ||
arguments or return values from one type to another: | ||
|
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.
Maybe link the corresponding explanation?
doc/Type/Metamodel/CoercionHOW.pod6
Outdated
Coercions L<pretend to be|/type/Metamodel::TypePretense> of types C<Any> | ||
and C<Mu>, and L<delegate|/type/Metamodel::MethodDelegation> methods to | ||
C<Any>. | ||
|
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.
Maybe provide an use case for this?
|
Yes that would be fine; maybe create a ticket to track any remaining work. |
Of the three, only Metamodel::CoercionHOW seems to be spec'd. |
7e55d2e
to
7f6f2c7
Compare
|
The problem
Metamodel::TypePretense
,Metamodel::MethodDelegation
, andMetamodel::CoercionHOW
are undocumented. While documentation for the latter isn't terribly important IMO, the former two are very useful when implementing your own HOWs and really need documentation. Documenting them also helps explain some behaviours of roles that may not be entirely clear.Solution provided
Document the metaroles and metaclasses mentioned, and update roles' HOWs' documentation.