Conversation
|
Just noticed that a test is failing and am a bit lost on how this could be correctly fixed.. There is a test that asserts that In this code snippet, getting the file timestamps for a file with wrong casing returns Need some help in figuring out what the right way to approach it is, until then I need to pause on this issue. |
alexander-akait
left a comment
There was a problem hiding this comment.
I don't think it is the valid fix, some OS has case-insensitive logic, same case-sensivity, it should be under an option (or we should check it using node.js API, not sure we can check it). Why your files have different cases, it is a not good practice (even more - there are plugins and rules for linter to prevent such things). But I agree it should be considered as a bug
Fixes the bug described in #228.
Verified by checking that the example posted in the original issue works now,
nullis not returned, and by adding a test that failed before the fix.