-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
配置文件合并 #892
Merged
Merged
配置文件合并 #892
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
After this commit, user_config.ini contains modified entries only
好的优化,验证通过。建议上述操作与注意项补充更新至教程文档说明 ( |
这两天在出差,我要是有空就更一下文档 |
文档已更新 |
麻烦辛苦在创建user_config.ini步骤中也补充一下这个提示 |
提示已补充 |
我提了两个阅读体验上的建议,辛苦你修改一下 |
你好,上述Codereview是否有空处理一下? |
我看了一下,目前的文档我认为已经写的很明白了,如果你有其他优化的想法可以直接推送到这个分支,或者本地checkout之后优化一下。如果是为了小白使用,还是维持复制粘贴的形式,毕竟什么是ini配置节都可能不懂。 |
已修改 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What kind of change does this PR introduce? (check at least one)
Which environment is this PR for? (check at least one)
Does this PR introduce a breaking change? (check one)
If yes, please describe the impact and migration path for existing applications:
If adding a new feature, the PR's description includes:
first and wait for approval before working on it)
Other information:
目前配置文件需要完全复制默认文件,更改配置项需要删除重建,如果配置未及时更新,会导致难以排查的bug。
修改后user_config.ini 文件只需要存储修改的配置项,注意ini section需要保持一致,user_config.ini覆盖config.ini中同section,同id的配置。
现有程序更新后,user_config.ini可只保留用户定制项,其余项删除。已经存在的配置由于是复制的默认文件,不修改也不会影响功能。