Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.6k
[VarDumper] Add support for adding more default casters toAbstractCloner::addDefaultCasters()#60480
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:7.4
Are you sure you want to change the base?
Conversation
AbstractCloner::addDefaultCasters()AbstractCloner::addDefaultCasters()AbstractCloner::addDefaultCasters()AbstractCloner::addDefaultCasters()Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Hello everyone. Thanks for the review! I have addressed your comments. |
| /** | ||
| * Adds default casters for resources and objects. | ||
| * | ||
| * Maps resources or objects types to a callback. |
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.
| * Maps resourcesorobjects types to a callback. | |
| * Maps resourcesorobject types to a callback. |
| * Adds default casters for resources and objects. | ||
| * | ||
| * Maps resources or objects types to a callback. | ||
| * Types are in the key, with a callable caster for value. |
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.
| *Types are in the key, with acallablecaster for value. | |
| *Usetypesas keys andcallablecasters asvalues. |
| * | ||
| * Maps resources or objects types to a callback. | ||
| * Types are in the key, with a callable caster for value. | ||
| * Resource types are to be prefixed with a `:`, |
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.
| *Resource typesare to be prefixedwitha `:`, | |
| *Prefix types with`::`, |
This PR let us to write better code, and we also get a better discovery