These are chat archives for Antaris/RazorEngine

16th
Dec 2015
Paul Cox
@coxp
Dec 16 2015 12:28
Are we using C#6 in RazorEngine going forward?
Matthias Dittrich
@matthid
Dec 16 2015 14:38
I don't see any reason why we shouldn't, we can use C#6 to compile for .net40 as far as I know...
Paul Cox
@coxp
Dec 16 2015 15:36
I'm not sure how well it works in VS 2013 but with VS 2015 Community edition and VS Code being free I'm not sure that is a valid reason not to use it.
Paul Cox
@coxp
Dec 16 2015 15:50
My spike isn't going very well trying to brute-force the separation of the MS Razor bits from RazorEngine.Core :(. I think it needs a bit more thought as it is quite tightly coupled to MS Razor throughout the code base. Theoretically I think only the "template"->"source code" stage of the pipeline should depend on MS Razor? I also think I need to separate the configuration into configuration of our pipeline and configuration of the individual view engines (e.g. we could still support CodeInspectors in Razor2 & 3 then as it would be part of their configuration not the global configuration). As this change would force some configuration I'm not sure if it would be worth keeping the static Engine.Razor service.
Matthias Dittrich
@matthid
Dec 16 2015 19:21
Yeah I tried as well and had a similar experience (see the split_api branch). Theoretically, yes you are correct. However there is a hidden dependency after we load the generated assembly and load the instance - as then we again need to have the type information for the 'TemplateBase' class which might depend on the MS Razor version... About the static class I'm not really sure if we need it (but a lot of users are used to use the library like this...). Maybe I shouldn't have shipped it with the new API, but I wanted to have an easy upgrade path.