Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
beginor opened this issue Aug 24, 2021 · 7 comments
Open

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

beginor opened this issue Aug 24, 2021 · 7 comments
Assignees

Comments

@beginor
Copy link
Collaborator

beginor commented Aug 24, 2021

这个项目目前使用的还是 .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 beginor mentioned this issue Nov 24, 2021
@beginor
Copy link
Collaborator Author

beginor commented Nov 24, 2021

@XiaoHeitu
Copy link
Contributor

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

@beginor
Copy link
Collaborator Author

beginor commented Jan 11, 2022

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

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

@Ahoo-Wang
Copy link
Member

Ahoo-Wang commented Jan 11, 2022

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

@Ahoo-Wang
Copy link
Member

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

@beginor
Copy link
Collaborator Author

beginor commented Jan 11, 2022

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

@Ahoo-Wang
Copy link
Member

Ahoo-Wang commented Jan 11, 2022

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

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

No branches or pull requests

3 participants