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

enhance: gf gen ctrl with -merge option #3490

Merged
merged 7 commits into from
Apr 18, 2024

Conversation

wln32
Copy link
Member

@wln32 wln32 commented Apr 16, 2024

  1. 改进:使用gen ctrl -merge参数时,可以一次将所有待生成的ctrl写入到文件,而不是分次写入。

  2. 添加两个测试分支,均使用 -merge 参数
    1:将 Req/Res 结构体追加到现有文件末尾
    2:将 Req/Res 结构体添加到新文件

@wln32 wln32 requested review from oldme-git, hailaz and gqcn and removed request for oldme-git April 16, 2024 03:36
@wln32 wln32 added CLI about gf cli enhancement labels Apr 16, 2024
@wln32 wln32 marked this pull request as draft April 16, 2024 04:29
wln added 3 commits April 16, 2024 12:39
1: Add the Req/Res structure to the existing file
2: Add the Req/Res structure to the new file
@wln32 wln32 marked this pull request as ready for review April 16, 2024 07:11
cmd/gf/internal/cmd/genctrl/genctrl_generate_ctrl.go Outdated Show resolved Hide resolved
cmd/gf/internal/cmd/genctrl/genctrl_generate_ctrl.go Outdated Show resolved Hide resolved
cmd/gf/internal/cmd/genctrl/genctrl_generate_ctrl.go Outdated Show resolved Hide resolved
@gqcn gqcn changed the title Improved: Optimized call flow when using gen ctrl -merge enhance: gf gen ctrl with -merge option Apr 16, 2024
@wln32 wln32 marked this pull request as draft April 16, 2024 14:02
@wln32 wln32 marked this pull request as ready for review April 17, 2024 04:06
@gqcn gqcn added the ready to merge Used in PR, which means this PR is reviewed. label Apr 17, 2024
@gqcn gqcn merged commit a39e6a2 into gogf:master Apr 18, 2024
23 checks passed
@wln32 wln32 deleted the Improve/cli-genctrl-merge branch April 19, 2024 05:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLI about gf cli enhancement ready to merge Used in PR, which means this PR is reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants