lsd-rs/lsd

Add `ls_colors` option to theme

オープン

#608 opened on 2022/01/17

GitHub で見る
 (2 件のコメント) (4 件のリアクション) (0 人の担当者)Rust (434 件のフォーク)batch import
area/themegood first issuekind/enhancement

Repository metrics

Stars
 (13,400 個のスター)
PR merge metrics
 (平均マージ 34d 3h) (30d で 2 merged PRs)

説明

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

コントリビューターガイド

調査方針
lsdのソースコード内の既存のテーマ設定構造を調べ、LS COLORS環境変数が現在どのように使用されているかを理解し、テーマ設定にオプションの 'ls colors' フィールドを実装して、環境変数を上書きできるが、フォールバックするようにします。
技術スタック
rust
領域
cli
Issue 種別
機能
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
半日
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
アクティブ
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
おおむね明確
前提条件
Rust
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
70