lsd-rs/lsd

Add `ls_colors` option to theme

Aperta

#608 aperta il 17 gen 2022

Vedi su GitHub
 (2 commenti) (4 reazioni) (0 assegnatari)Rust (434 fork)batch import
area/themegood first issuekind/enhancement

Metriche repository

Star
 (13.400 stelle)
Metriche merge PR
 (Merge medio 34g 3h) (2 PR mergiate in 30 g)

Descrizione

As of now, we support theme for colors, and still use ls_colors for file name colors, how about adding the ls_colors option to the theme configuration file, and so that we can configure all the colors in one place.

also, there is something that needs to be covered:

  1. the ls_colors in the theme should be optional, and fall back to LS_COLORS env
  2. the one in themes should overwrite the env one

@meain

Guida contributor

Direzione di ricerca
Esamina la struttura di configurazione del tema esistente nel codice sorgente di lsd, capisci come viene attualmente utilizzata la variabile d'ambiente LS COLORS e implementa un campo 'ls colors' opzionale nella configurazione del tema che possa sovrascrivere la variabile d'ambiente ma ricadere su di essa.
Tech stack
rust
Dominio
cli
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
Rust
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70