zeromicro / go-zero

A cloud-native Go microservices framework with cli tool for productivity.
https://go-zero.dev
MIT License
29.38k stars 3.97k forks source link

goctl template --home 指定自定义模板目录不生效 #1330

Closed leafney closed 2 years ago

leafney commented 2 years ago

我还是用中文描述吧!

Describe the bug

goctl release v1.2.4 使用 --home 指定自定义模板目录不生效 版本:

➜ goctl --version
goctl version 1.2.3-cli darwin/amd64

To Reproduce Steps to reproduce the behavior, if applicable:

  1. The code is

    ➜ goctl api go -api="test.api" -dir="api" -style GoZero --home ./template/1.2.3-cli
  2. The error is

我修改了默认模板中 api 的 handler 和 logic 模板 api/handler.tplapi/logic.tpl ,通过上面的命令生成后发现还是使用的默认自带的模板。 更具体一点,我是参照 https://go-zero.dev/cn/template.html 这个例子,更改了 handler 中的返回格式 将 httpx.OkJson(w, body) 改成了 reponse.Response(w, resp, err) 这种。

开始我以为是目录的问题,然后我就放到了 ~/.goctl 下面,测试发现还是不生效。

后来我又换回了之前使用的 goctl v1.2.1 , 发现这个是生效的。所以怀疑是 v1.2.4 有问题。

Expected behavior A clear and concise description of what you expected to happen.

Environments (please complete the following information):

More description Add any other context about the problem here.

chensylz commented 2 years ago

so do I

kesonan commented 2 years ago

感谢反馈,这个问题已经发现了,目前我们已经在讨论这个解决方案,目前的解决方案是在通过 --home 指定的目录下需要创建一个与 goctl 版本号的目录来存放你的模板,这样 --home才能生效,如你的 goctl 版本为 1.2.3, 你的自定模板版本应该在 path/to/custom/1.2.3/templates, 那么 --home 指定的值应该为 path/to/custom

leafney commented 2 years ago

突然有一个想法,是否可以根据goctl的当前版本来自动判断加载哪个目录。或者是,将版本作为一个参数传入。 当然,不清楚你们这样设计的目的是为什么解决什么问题。只是觉得对于开发者来说,在使用上还是越简单越好,可能设计就会比较复杂一些。 谢谢!

pengmingfong commented 5 months ago

这个问题在哪个版本解决掉了呀?我go-zero 1.6.3 版本还是有这个问题

Issues-translate-bot commented 5 months ago

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑‍🤝‍🧑👫🧑🏿‍🤝‍🧑🏻👩🏾‍🤝‍👨🏿👬🏿


In which version has this problem been solved? I still have this problem in go-zero 1.6.3 version

pengmingfong commented 5 months ago

@kesonan 这个问题在哪个版本解决掉了呀?我go-zero 1.6.3 版本还是有这个问题

Issues-translate-bot commented 5 months ago

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑‍🤝‍🧑👫🧑🏿‍🤝‍🧑🏻👩🏾‍🤝‍👨🏿👬🏿


@kesonan In which version has this problem been solved? I still have this problem in go-zero 1.6.3 version