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.

Sign up for GitHub

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

Jump to bottom

venus-market 初始化时, 将piecestore(或者其它)比较复杂的配置以注释方式输出到配置文件中. #5036

Closed
1 task done
Tracked by #5125
zl03jsj opened this issue Jul 6, 2022 · 2 comments
Closed
1 task done
Tracked by #5125

venus-market 初始化时, 将piecestore(或者其它)比较复杂的配置以注释方式输出到配置文件中. #5036

zl03jsj opened this issue Jul 6, 2022 · 2 comments
Labels
P1 High - we should be working on this now or in the immediate future V-venus-market venus-market component

Comments

@zl03jsj
Copy link
Contributor

zl03jsj commented Jul 6, 2022 •
edited
Loading

Venus component

  • venus market

What is the motivation behind this feature request? Is your feature request related to a problem? Please describe.

对于这些比较复杂的类型, 如果没有样例, 的确很难编写其配置.

Describe the solution you'd like

初始化时, 将这一部分复杂的配置写入到配置文件的注释中, 后续就可以参考注释来提供对应的配置方式.

Describe alternatives you've considered

none

Additional context

none
@zl03jsj zl03jsj added P1 High - we should be working on this now or in the immediate future V-venus-market venus-market component labels Jul 6, 2022
@zl03jsj zl03jsj self-assigned this Jul 6, 2022
@LinZexiao
Copy link
Collaborator

另见 #4906

@diwufeiwen diwufeiwen mentioned this issue Jul 25, 2022
Closed
36 tasks
@LinZexiao
Copy link
Collaborator

目前，我们当前使用的toml编码库 https://github.com/BurntSushi/toml 暂时还不支持插入注释这一的特性
另外 配置文档也从另一个维度解决了这个问题。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
P1 High - we should be working on this now or in the immediate future V-venus-market venus-market component
Projects
Venus Project
Archived in project
Milestone
No milestone
Development

No branches or pull requests
2 participants
@zl03jsj @LinZexiao