Skip to content
This repository has been archived by the owner on Feb 23, 2021. It is now read-only.

Change TagHelperDescriptor resolution to be invoked via a Program.Main. #25

Closed
NTaylorMullen opened this issue Sep 17, 2015 · 2 comments

Comments

@NTaylorMullen
Copy link
Member

Today we rely on the DTH to instantiate the RazorPlugin which is then pumped messages. Due to several limitations we've decided to move this logic into a separate process outside of the DTH.

Therefore we should be able to pass in command line arguments to then determine TagHelperDescriptors. Will also need to support protocols.

@Eilon Eilon added this to the 1.0.0-rc1 milestone Sep 17, 2015
@Eilon Eilon modified the milestones: 1.0.0-beta8, 1.0.0-rc1 Sep 18, 2015
@NTaylorMullen NTaylorMullen modified the milestones: 1.0.0-beta8, 1.0.0-rc1 Sep 24, 2015
NTaylorMullen added a commit that referenced this issue Sep 29, 2015
- This is a half-baked conversion to executing the `RazorPlugin`'s `TagHelperDescriptor` resolution pieces as an executable. We don't want to fully convert to using a `Program.Main` until we have tooling support.

#25
NTaylorMullen added a commit that referenced this issue Oct 1, 2015
- This is a half-baked conversion to executing the `RazorPlugin`'s `TagHelperDescriptor` resolution pieces as an executable. We don't want to fully convert to using a `Program.Main` until we have tooling support.

#25
@NTaylorMullen NTaylorMullen modified the milestones: 1.0.0-rc2, 1.0.0-rc1 Oct 26, 2015
@NTaylorMullen
Copy link
Member Author

This issue should be treated as "remove the ability for the Razor tooling package to be invoked via DTH plugins".

NTaylorMullen added a commit that referenced this issue Dec 9, 2015
- This resulted in us decoupling ourselves from the `IPlugin` interface. It's going away.
- Redesigned the `TagHelperDescriptor` resolver to be a glorified factory with a protocol. It used to serialize/deserialize messages as input because it was an `IPlugin`.

#25
@NTaylorMullen
Copy link
Member Author

3b920e0
e33e929

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants