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

goctl specifications and recommendations #3805

Open
sunviv opened this issue Dec 19, 2023 · 7 comments
Open

goctl specifications and recommendations #3805

sunviv opened this issue Dec 19, 2023 · 7 comments

Comments

@sunviv
Copy link

sunviv commented Dec 19, 2023

Could you unify the usage specifications of the goctl?
goctl model mysql ddl --src user.sql --dir .
goctl api go --api user.api --dir .
Why not change the input files to --src and the output files to --dir

It is recommended to add the function of modifying the generated file name
image
In the logic directory, I don't want the generated filenames to contain '_logic'.

@fuckqqcom
Copy link

@sunviv 看我fork的一个版本,我目前都去掉了这些,
image

@kesonan
Copy link
Collaborator

kesonan commented Dec 22, 2023

Thanks for the feedback. In the early, goctl was developed by multiple people, each with the same hobby. At first, everyone focused on solving problems. When these problems were discovered, it was too late to modify them.

@sunviv
Copy link
Author

sunviv commented Dec 27, 2023

Thanks for the feedback. In the early, goctl was developed by multiple people, each with the same hobby. At first, everyone focused on solving problems. When these problems were discovered, it was too late to modify them.

And how to modify the file name generated by goctl?

@fuckqqcom
Copy link

@kesonan 有打算给types支持分组的计划嘛

@Issues-translate-bot
Copy link

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


@kesonan Do you have any plans to support grouping for types?

@kesonan
Copy link
Collaborator

kesonan commented Jan 3, 2024

@kesonan 有打算给types支持分组的计划嘛

There is no plan support, the API does not have the concept of packages, grouping will bring negative effects and duplicate code

@Issues-translate-bot
Copy link

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


@sunviv go-sh@9da3b68 You can try this branch

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

4 participants