You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
-
1. 输出sample文件
目前: api -o 和 rpc template -o
建议: 统一为 api -o 和 rpc -o
相关issue或pr: #1684
2. goctl api 和 goctl rpc 都不加参数应该怎样 [solved]
目前行为如下:
因为都有子命令,建议直接显示help信息,或者都显示missing -o,第一个issue合并之后也是
rpc -o
输出sample文件3. api new 和 rpc new 行为要统一 [solved]
目前行为如下:
都不加参数的时候,api new 会默认生成一个 greet服务,rpc报错,并且生成一个.proto文件
通常像new这种命令,不加参数的时候: 1. 要么给出错误,提示用户需要显示指定一个名字 2. 要么直接显示help信息 3. 要么以运行命令所在的目录名为参数
相关issue或pr: #1688 #1687
4. goctl docker without flags [solved]
goctl model, goctl rpc, goctl template
这些命令都有子命令所以默认会显示help信息
虽然docker没有子命令,不加参数建议也处理下,直接显示显示help信息
相关issue或pr: #1677
5. rpc protoc
这个算是goctl里最复杂的命令了 很多新手卡在这里,要么环境配置的问题,要么不知道怎么调用,这个迟点再讨论
#1683 #1716
6. update api gen go logic [hold on]
#1695
https://talks.golang.org/2014/names.slide#10
Return values on exported functions should only be named for documentation purposes.
7. timestamp fields [WIP]
// ......................
create_time -> created_at
update_time -> updated_at
相关issue或pr: #1707
solution: see #1795
建议: 这个会有breaking change, 做兼容还得加很多逻辑判断,不符合从简的原则,建议就不先不管了,或许以后有breaking change的版本发布的时候再考虑merge
8. default format for --style should be go_zero [hold on]
相关issue或pr: #1692
个人觉得对于文件名,service_context.go shorten_logic.go wire_gen.go greet_handler.go 这种比去掉下划线的更可读
当然effective go 和 uber go style guide 都没有规定文件名格式。但是我搜索了下golang和kubernetes里的源码,带下划线的居多
golang/go#36060 (comment)
Beta Was this translation helpful? Give feedback.
All reactions