Jake Bailey

Results 1405 comments of Jake Bailey

I need to think about this harder; I have to go manually merge this into one of my branches to get it to test so it's annoying to figure out.

No, this does seem to work, I just merged the two PRs together wrong.

> I noticed that when I re-ran the generation, I got different results than the currently committed file This shouldn't be possible; the script is deterministic. Did you run the...

Merging #789 and rerunning CI here shows that the formatting is in fact being changed in an unexpected way.

I think I'm going to wait for the other two user pref related PRs to finish and just fix this PR instead of forcing that on everyone.

One last question; these entrypoints are not documented by the package at all. Are they intended to be imported like this?

Is there a test you can make for this? The fix seems obviously right, however. The original code looked more like: ```ts case MatchingMode.Pattern: const starPos = pathOrPattern.indexOf("*"); const leadingSlice...

> Interestingly the original code looks like it may be subtly incorrect as well. It won't crash due to JS's more lenient slice() but it does have the false positive...

Per the linked issue, this only works if downstream users use TS 5.6+, right?