Record frame: remaining review nits (PR #147)
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
Read the listed code locations in RecordFileReader.cs, RecordRoot.cs, and RecordJsonSchema.cs, then review reports/te-review-record-frame.md and the fixtures under records/. Start by separating the parser, validation, schema, fixture, and depth-limit items into independently scoped changes. Done means each selected review nit has a focused fix and regression coverage; the issue lists several unrelated items, so a first contributor should coordinate which one to take.
索引モデルが issue の本文から書いたものです。
説明
From H:\work\gdr-design-20261102\reports\te-review-record-frame.md P3s not fixed in the PR:
- A tab or several spaces after key: gets no scalar-style finding, so a rewrite changes bytes silently.
- A raw tab in a plain-text block reads as a space with only an info finding (RecordFileReader.cs:777-786).
- RecordRoot.cs:30-40: /objects/ accepted as a whole record of the file's kind; top folder never checked.
- RecordJsonSchema.cs:23 hardcodes fwmd 1; :70 '--frame' definition name can collide.
- RecordFileReader.cs:855: a notation fix with several edits is dropped silently.
- Fixtures: reused ids (fwmd.ws.unknown, fwmd.syntax.unknown-attribute, fwmd.env.unknown-class) have no ok: negatives in records/; more syntax-error cases.
- Watch: YamlDotNet takes 1.8 s on an unclosed [ nested 20,000 deep; consider a depth cap.
- 主要言語
- C#
- スター
- 0
- フォーク
- 0
- 平均マージ
- 1時間 43分
- マージ済み PR（30日）
- 43
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
sillsdev/SIL.TextEditing のほかの issue
-
Playground selection bar: comes back after Apply or scrolling while focus is in the Markdown paneオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
sillsdev/SIL.TextEditing#175 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
sillsdev/SIL.TextEditing#167 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
sillsdev/SIL.TextEditing#164 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
sillsdev/SIL.TextEditing#137 ·
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1〜3時間 初心者へのやさしさ 82/100
sillsdev/SIL.TextEditing#133 ·
メンテナーはふだん 1 日以内に返信
sillsdev/SIL.TextEditing の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 77/100
BEagle1984/silverback#280 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 63/100
-
Status: Needs Triage
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
PSAppDeployToolkit/PSAppDeployToolkit#2392 ·
メンテナーはふだん 1 日以内に返信
-
needs attention untriaged
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet#3666 ·
メンテナーはふだん 3 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 73/100
MicrosoftLearning/eShopOnWeb#466 ·