Skip to content
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

[resolvers]: Delegates as resolvers #1584

Merged
merged 6 commits into from
Mar 19, 2023
Merged

Conversation

pekkah
Copy link
Owner

@pekkah pekkah commented Mar 19, 2023

  • Add Delegate as resolver
  • Delegate can take ResolverContext argument
  • Delegate can take any property of ResolverContext as argument (by name, ignores case)
  • Delegate can take additional arguments which are resolved from ResolverContext.RequestServices
  • Delegate return value can be void, ValueTask, Task
  • Delegate return value can be T, ValueTask, Task and the returned value is assigned to ResolverContext.ResolvedValue automatically

@pekkah pekkah enabled auto-merge (squash) March 19, 2023 18:15
@pekkah pekkah disabled auto-merge March 19, 2023 18:15
@pekkah pekkah enabled auto-merge (squash) March 19, 2023 18:19
@pekkah pekkah merged commit 6398415 into master Mar 19, 2023
@pekkah pekkah deleted the feature/delegate-resolver branch March 19, 2023 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant