Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update fastwalk to v1.0.8 for better MSYS detection and sorting #3930

Merged
merged 1 commit into from
Jul 19, 2024

Conversation

charlievieth
Copy link
Contributor

@charlievieth charlievieth commented Jul 18, 2024

NOTE: This PR needs to be updated to not use the slices package.

This commit updates github.com/charlievieth/fastwalk to v1.0.7 which improves MSYS detection and adds optional sorting of directory entries. It also updates fzf to use the SortFilesFirst sort mode which improves the output by making it a bit more sorted and grouped by directory previously entries were visited in directory order (which is basically random). The performance impact of sorting is minimal compared to the cost of the syscalls needed to read directories.

PRs Included:

@charlievieth charlievieth marked this pull request as draft July 18, 2024 04:32
This commit updates github.com/charlievieth/fastwalk to v1.0.8 which
improves MSYS detection and adds optional sorting of directory entries.
It also updates fzf to use the SortFilesFirst sort mode which improves
the output by making it a bit more sorted and grouped by directory
previously entries were visited in directory order (which is basically
random).

PRs Included:

  * charlievieth/fastwalk#25
  * charlievieth/fastwalk#27
  * charlievieth/fastwalk#28
@charlievieth charlievieth marked this pull request as ready for review July 18, 2024 15:06
@charlievieth charlievieth changed the title Update fastwalk to v1.0.7 for better MSYS detection and sorting Update fastwalk to v1.0.8 for better MSYS detection and sorting Jul 18, 2024
@junegunn
Copy link
Owner

Thanks! SortFilesFirst is indeed a nice improvement.

@junegunn junegunn merged commit a2c365e into junegunn:master Jul 19, 2024
5 checks passed
@junegunn
Copy link
Owner

Just released a new version with the updated dependency. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants