Taiwon Chung

Results 13 comments of Taiwon Chung

Internal Ref: GO-1736

Could you provide a specific repro for this issue?

Hey, sorry I missed this. Yeah, I think making it less verbose could be a valid improvement on this (as a syntactic sugar), but meanwhile we can probably close this...

This has been completed with #990.

I agree with @JacobOaks that this is probably fine, but we should clearly document that those values in `CallbackInfo` shouldn't be modified.

Before we add a whole new top-level API for registering a global callback for all providers, could you consider adding the constructed values into CallbackInfo? I think it would be...

@cebilon123 You can submit a PR if you have a fix. We welcome any contribution.

I agree with @JacobOaks that skipping internal callers could be better.

I'm not really sure what kind of benefit this brings. Could you explain more in detail why this feature is needed?

Thanks for the detailed explanation! I'll bring this up to the team.