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

Add --no-git option #1239

Closed
wants to merge 1 commit into from
Closed

Add --no-git option #1239

wants to merge 1 commit into from

Conversation

Aehmlo
Copy link

@Aehmlo Aehmlo commented Aug 19, 2023

Some unnecessary exposition about my use case

While updating my home-manager configuration recently, I discovered that exa now has a program module available, with the option programs.exa.git to alias exa to exa --git. I happily enabled this option but quickly found myself in situations where I didn't want the git status to shown. While I could use something like $(which exa) -l, I far prefer the ergonomics of a --no-git option (and it plays more nicely with other tools to boot).

This PR adds the --no-git option. This option overrides --git in all cases and disables showing the Git status of files in long view (-l). This enables users of shell aliases and the like to turn off git status display in an ad hoc manner

Somewhat tangentially related: #1044.

(This is my first contribution to this codebase; I've done my best to make all the appropriate changes, but I'd appreciate any pointers if there's something I've missed. I took inspiration from the --no-icons option as prior art.)

This option overrides --git in all cases and disables showing the Git
status of files in long view (-l).
@cafkafk
Copy link

cafkafk commented Aug 26, 2023

Hey, fyi, exa is super dead, but if you'd like to, consider submitting this pr to eza.

@Aehmlo
Copy link
Author

Aehmlo commented Aug 26, 2023

@cafkafk Thanks for the head-up. I just created eza-community/eza#168.

@ariasuni
Copy link
Collaborator

ariasuni commented Sep 6, 2023

Closing this since exa is unmaintained (see #1243), and this PR has been merged in the active fork eza.

@ariasuni ariasuni closed this Sep 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants