Better cope with PowerShell sending a BOM (Windows), or stop trying
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 35/100
調査の方向性
src/SeqCli/PlainText/Framing/FrameReader.cs の 39 行目から始め、Windows 上の PowerShell 5.1 に対する既存の BOM 回避策を理解してください。可能であれば動作を再現し、PowerShell Core 6.2 と比較して、maintainer がどのオプションを望んでいるかを確認してください。選択した動作が実装され、影響を受けるコマンドライン入力経路が検証されれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Version 5.1.213 includes a small hack to try to get around encoding issues when running seqcli under PowerShell on Windows: https://github.com/datalust/seqcli/blob/dev/src/SeqCli/PlainText/Framing/FrameReader.cs#L39
Unfortunately, this is racy, and doesn't work all the time.
Our options are:
- Dig deeper to figure out how it is that the BOM gets through, even with a UTF-8 encoding specified for STDIN
- Try to skip the BOM in more places (when we read the first line in
FrameReader - Stop trying
I think we're better off giving up; PowerShell Core no longer has this issue (verified with 6.2), so it's a point-in-time thing anyway.
- 主要言語
- C#
- スター
- 167
- フォーク
- 30
- 平均マージ
- 1日 6時間
- マージ済み PR（30日）
- 9
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
datalust/seqcli のほかの issue
-
enhancement
難易度 3/5 1〜2日 初心者へのやさしさ 45/100
-
enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 20/100
-
enhancement
難易度 4/5 3〜5日 初心者へのやさしさ 38/100
-
enhancement
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
-
enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
datalust/seqcli の issue をすべて見る
似ている issue
-
untriaged
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
dotnet/dotnet-api-docs#13095 ·
-
area-deployment area-integrations triage:bot-seen
難易度 2/5 半日 初心者へのやさしさ 86/100
-
type/automation type/tech-debt
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
-
bug
難易度 1/5 1時間未満 初心者へのやさしさ 90/100
newrelic/newrelic-dotnet-agent#3850 · コメント 1 件 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
LuckyPennySoftware/AutoMapper#4660 ·