lsd-rs / lsd-rs/lsd

config support set display to VisibleOnly, but CLI arg not

Open
#451 0 comments 0 reactions 0 assignees View on GitHub
kind/enhancement
Dominant language
Rust
Stars
16.2k
Forks
510
PR merge metrics
No merged PRs in 30d

Description

after https://github.com/Peltoche/lsd/pull/443#discussion_r535595544 merged, we can config the `display` to `VisibleOnly`, but not CLI args.
there will be a scenario like
1. set `display` in config file to value other than `VisibleOnly` for example `all`
2. users want to temporarily set `display` to `VisibleOnly`

users will have to change the configuration in config file, can not use a CLI arg.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reviewing pull request #443 and its linked discussion, then trace how the config-file `display` value and CLI arguments are handled. Done means a CLI argument can temporarily select `VisibleOnly` when the configuration sets another display value.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
cli
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.