fix(#3117): allow changing filename's casing #3118
Conversation
There was a problem hiding this comment.
I've tested the new code on linux and it executes as per the existing, for what that's worth.
- Have you tested on powershell?
It's not worth checking errors from the fs_ functions; we cannot reasonably action any problems, matching existing code. Checkting nil returns is sufficient.
As this is behind a feature flag, I'm happy to merge it. If there are any issues we'll revert it and call on you to make a fix, as we lack the OS and knowledge to do so.
There was a problem hiding this comment.
I've tested the new code on linux and it executes as per the existing, for what that's worth.
- Have you tested on powershell?
- Are there any other test cases e.g. command.com?
It's not worth checking errors from the fs_ functions; we cannot reasonably action any problems, matching existing code. Checkting nil returns is sufficient.
As this is behind a feature flag, I'm happy to merge it. If there are any issues we'll revert it and call on you to make a fix, as we lack the OS and knowledge to do so.
|
Good call, breaks on Windows. Will look into it and fix. |
Closes (#3117). Seems a little crude, but I don't think there is an easier way.