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

1.upgrade rust to 1.85.0. 2. add wildmatch to the path #1393

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

awol2005ex
Copy link

1.upgrade rust to 1.85.0. 2. add wildmatch to the path

Copy link
Member

@gierens gierens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The wildmatch thing looks interesting, thanks, but could you please explain how this exactly improves things, is it about performance or about behavior and a specific example where it improves it. Also a short performance comparison with hyperfine would be nice.

About the MSRV change: The reason we are trailing Rust stable by several versions as many other projects like bat do as well, is that not all systems can install the latest rust version via rustup for example and thus would not be able to build eza if we would use the latest version. Also, I saw you opened a similar PR for bat: sharkdp/bat#3216 . So while I would lean towards no, we are not upgrading the MSRV to 1.85 now, I would also like to hear first what the bat team has to say about this.

Last but not least, while this PR should actually be two PRs, it should at least split the different things (wildmatch and msrv) into separate commits that also follow conventional commit format.

@awol2005ex
Copy link
Author

The wildmatch thing looks interesting, thanks, but could you please explain how this exactly improves things, is it about performance or about behavior and a specific example where it improves it. Also a short performance comparison with hyperfine would be nice.

About the MSRV change: The reason we are trailing Rust stable by several versions as many other projects like bat do as well, is that not all systems can install the latest rust version via rustup for example and thus would not be able to build eza if we would use the latest version. Also, I saw you opened a similar PR for bat: sharkdp/bat#3216 . So while I would lean towards no, we are not upgrading the MSRV to 1.85 now, I would also like to hear first what the bat team has to say about this.

Last but not least, while this PR should actually be two PRs, it should at least split the different things (wildmatch and msrv) into separate commits that also follow conventional commit format.

I want to do like that : eza -l E:\workspace* and show the folder like E:\workspace_node ,E:\workspace_java ,E:\workspace_rust ....

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

Successfully merging this pull request may close these issues.

2 participants