Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

fix: gf gen dao input param and config file priority #3527

Closed
wants to merge 2 commits into from
Closed

fix: gf gen dao input param and config file priority #3527

wants to merge 2 commits into from

Conversation

oldme-git
Copy link
Member

@oldme-git oldme-git commented Apr 22, 2024 •
edited
Loading

Dear review:
gf gen dao 命令会优先使用配置文件中的数据。
例如，键入 gf gen dao -link=... ，则不会命令中的 link 参数，而是继续使用配置文件。这不太合理，现在优化成这样：
以配置文件为底，如果检测到额外的参数，则合并到配置文件中的数据去生成 dao

Fixed #3459

@oldme-git oldme-git changed the title fix: 3459 fix: gf gen dap input param and configuration file priority Apr 22, 2024
@oldme-git oldme-git changed the title fix: gf gen dap input param and configuration file priority fix: gf gen dao input param and configuration file priority Apr 22, 2024
@oldme-git oldme-git changed the title fix: gf gen dao input param and configuration file priority fix: gf gen dao input param and config file priority. Apr 22, 2024
@oldme-git oldme-git changed the title fix: gf gen dao input param and config file priority. fix: gf gen dao input param and config file priority Apr 22, 2024
@oldme-git oldme-git added the wip label Apr 22, 2024
@oldme-git oldme-git closed this Apr 22, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
wip
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

gf gen dao link option doesn't work
1 participant
@oldme-git