feat(telescope): improve file_ignore_patterns

[KevinNitroG]

this will ignore node_module and .git folders in across OSes (Windows, Linux, Mac...) (I guess)

without this commit, it will ignore some folder that shouldn't be ignored (ex: .github/ on Windows)

Edit: I think the reason it ignores is Windows use \ as path separator.
Edit: Here is the telescope.defaults.file_ignore_patterns doc.

[Alexis12119]

I think telescope.nvim is smart enough to just use / instead of \\.

[Alexis12119]

For the node_modules you don't need to adjust it. Also, I didn't add a separator to really exclude them. But, if others need it then I'll probably add it.

[KevinNitroG]

I think telescope.nvim is smart enough to just use / instead of \\.

@Alexis12119 no ((: without \\, it will ignore too. It's like

---

I think for example, the .github/workflows/sth.yml path on windows it will be .github\workflows\sth.yml. And the pattern .git will match the path, so it will ignore the path.

If you only have the pattern .git/, the telescope isn't that smart as you thought and will won't match the path .github\workflows\sth.yml, also don't match the path of .git\ folder on Windows either.

And the result of only have .git/ is that it won't ignore either .git\ or .github\ on Windows.

---

Please use git revert if you want to revert the commit 😥

[KevinNitroG]

I tested it on windows too though

Huhm, In this video I use your current patterns.

2024-06-13.11-12-19.mp4

It won't ignore all files in .git\ folder in my case.

[Alexis12119]

I'll just add .git\\

[Alexis12119]

It works maybe before because I didn't add a separator, but it causes to exclude something we don't want.