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

fix(core): config-override from a directory #591

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from

Conversation

panzeyu2013
Copy link

@panzeyu2013 panzeyu2013 commented May 14, 2024

这个PR是为了解决问题 #590 在config-override中指定路径时无法正常初始化,必须指定为某个具体的可序列化文件的问题

由于在LoadFromFile中已经实现了初始化的部分,经检查问题在于在保存时会将保存路径+json文件名覆盖为指定路径,File无法对路径进行写入,故产生错误

修改后如果config-override指定为目录时,会自动在当前目录下生成config.json(但是会对原有的config进行覆盖),可以解决报错的问题

@Genteure
Copy link
Member

Genteure commented May 15, 2024

这个改动感觉不太好,因为理论上来讲之前它是支持使用任意文件后缀名的,这样改过之后就变成强制 json 了。
可以考虑用检测传入的路径是否是一个文件夹的方式来实现同时支持文件和文件夹。

@panzeyu2013
Copy link
Author

panzeyu2013 commented May 15, 2024

在第二个commit中的改动支持任意后缀名的文件,以及当指定路径为目录时在自动在当前目录下初始化一个config.json

还需要考虑的问题是是如果指定目录下已经存在之前建立的config.json文件,只在这个部分进行改进会导致对原有配置的覆盖,应该在Program.cs初始化调用的时候再加一个逻辑判断,等我再交一个commit

@panzeyu2013
Copy link
Author

现在好像应该没问题了

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants