search

gokcehan / lf

Terminal file manager
MIT License
7.8k stars 330 forks source link

Move get file extension logic to util #1670

Closed Limero closed 7 months ago

Limero commented 8 months ago

As a follow-up to @joelim-work's comment on this PR https://github.com/gokcehan/lf/pull/1664#pullrequestreview-1970078322

The changes that seem unrelated are automatic formatting by gofumpt, I can revert those if you don't want them applied

joelim-work commented 7 months ago

Just realized that this PR also fixes the bug described in https://github.com/gokcehan/lf/issues/387#issuecomment-645882460 with the following files:

.a/
a/
.a.txt
a.txt

I will update the PR description accordingly.