dotnetcore / SmartCode

SmartCode = IDataSource -> IBuildTask -> IOutput => Build Everything!!!
Apache License 2.0
572 stars 163 forks source link

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

Open beginor opened 3 years ago

beginor commented 3 years ago

这个项目目前使用的还是 .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 commented 2 years ago

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

XiaoHeitu commented 2 years ago

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

beginor commented 2 years ago

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

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

Ahoo-Wang commented 2 years ago

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

Ahoo-Wang commented 2 years ago

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

beginor commented 2 years ago

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

Ahoo-Wang commented 2 years ago

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