SmartCode icon indicating copy to clipboard operation
SmartCode copied to clipboard

升级 SmartCode 到 .Net 新版本的一些想法

Open beginor opened this issue 3 years ago • 7 comments

这个项目目前使用的还是 .Net Core 2.2 , 已经失去了官方的支持。 其中的原因, 已经在问题 #48 中讨论过了。

阻碍 SmartCode 继续升级最大的原因是使用了 Asp.Net Core Mvc 2.2 的视图引擎作为模板, 导致无法升级。

在我的 fork (https://github.com/beginor/smartcode)中, 已经将 Asp.Net Core Mvc 2.2 视图引擎替换为 https://github.com/adoconnection/RazorEngineCore , 还是 Razor , 还是 Cshtml , 语法也差不多, 但是不是 Asp.Net Core Mvc , 所以原来的模板无法使用, 所以就没有提交合并请求。

不知道还有多少人还再使用这个项目, 是否有人愿意一起来修改现有的模板呢, 凭我个人之力, 肯定无法完成全部模板的修改, 而且我不用这些模板。

这个项目能否继续发展, 需要大家一起努力。

简单说, 我可以提交合并请求, 把我的修改贡献出来,但是后果是现有的模板不能直接用, 需要同步修改全部模板。

beginor avatar Aug 24 '21 03:08 beginor

新的模板示例: https://github.com/beginor/smartcode/blob/master/src/SmartCode.Generator/RazorTemplates/Contributions/AppFx/NHibernate/Entity.cshtml

beginor avatar Nov 24 '21 09:11 beginor

需要我们怎么做,是在你的分支上修改模版提交PR,合并后再由你提交PR给官方吗?

XiaoHeitu avatar Jan 11 '22 06:01 XiaoHeitu

我已经提交了推送请求, @Ahoo-Wang 也说会尽快进行审阅, 但似乎一直没有任何动作。

@XiaoHeitu 如果你急用的话,也可以先在我的分支上提交 PR 。

beginor avatar Jan 11 '22 06:01 beginor

@beginor 目前没有合并到主分支的原因是会导致原可用的模板失效,我们先合并到 https://github.com/dotnetcore/SmartCode/tree/dotnet6 完成剩余的模板兼容后再合并到主分支?

Ahoo-Wang avatar Jan 11 '22 06:01 Ahoo-Wang

@beginor 如果觉得可行的话,可以将该PR https://github.com/dotnetcore/SmartCode/pull/64 先合并到 https://github.com/dotnetcore/SmartCode/tree/dotnet6

Ahoo-Wang avatar Jan 11 '22 06:01 Ahoo-Wang

当然没问题, 已经修改推送请求的目标分支。

beginor avatar Jan 11 '22 06:01 beginor

ok,已合并。接下来主要的工作是部分模板的兼容性问题,我们完成了这部分工作就可以合并到主分支了。 @beginor 我提升了你的权限,应该是可以执行Merge PR的。

Ahoo-Wang avatar Jan 11 '22 06:01 Ahoo-Wang