这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@jonsequitur
Copy link
Contributor

In the process of creating a Display helper, I've also started some refactoring work to remove the KernelPipelineContext and replace it with a hopefully more understandable KernelCommandBase.Handler method. Similar to the way ASP.NET Core middleware works, this Handler is assignable so that the kernel can make decisions about how to implement the given command. The handler has access to the KernelInvocationContext in order to raise events much like before.

I'm considering additional changes here to replace the OnNext / OnCompleted / OnError methods on KernelInvocationContext with something more specific to the use cases here.

@jonsequitur jonsequitur force-pushed the display-method branch 2 times, most recently from 672f4af to 9976794 Compare August 7, 2019 21:37
@jonsequitur jonsequitur changed the title [WIP] command handling simplification, Display helper command handling simplification, Display helper Aug 7, 2019
@jonsequitur jonsequitur merged commit 727401e into dotnet:master Aug 8, 2019
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.

2 participants