I've realised that the new LoadSettings(string) has the risk of being confused with the original LoadSettings method as all inputs to the original method are optional, and the first input to both is a string so to avoid confusion, I'm going to rename the new method to be LoadSettingsFromFile instead.
Thanks @Tom-Kingstone for pointing it out just as I hit the merge on #3300 😄
I've realised that the new
LoadSettings(string)
has the risk of being confused with the originalLoadSettings
method as all inputs to the original method are optional, and the first input to both is astring
so to avoid confusion, I'm going to rename the new method to beLoadSettingsFromFile
instead.Thanks @Tom-Kingstone for pointing it out just as I hit the merge on #3300 😄