Record frame: remaining review nits (PR #147)
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 35/100
Piste de recherche
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.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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.
- Langage dominant
- C#
- Étoiles
- 0
- Forks
- 0
- Merge moyen
- 1 h 43 min
- PR mergées (30 j)
- 43
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de sillsdev/SIL.TextEditing
-
Playground selection bar: comes back after Apply or scrolling while focus is in the Markdown paneOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
sillsdev/SIL.TextEditing#175 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
sillsdev/SIL.TextEditing#167 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
sillsdev/SIL.TextEditing#164 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
sillsdev/SIL.TextEditing#137 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 1-3 heures Accessibilité débutants 82/100
sillsdev/SIL.TextEditing#133 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de sillsdev/SIL.TextEditing
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
flyingpie/windows-terminal-quake#419 ·
Les mainteneurs répondent en général sous 1 jour
-
Show the per-model weekly limitOuverteenhancement help wanted
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
-
a-corriger documentation
Difficulté 1/5 1-3 heures Accessibilité débutants 83/100
christian-raj/S-Aloha#272 ·
-
Incorrect loop bound in SizeOptIListSelectIterator.ToListPeut-être pris @equmagg l’a pris aujourd’hui. Ouvertearea-System.Collections untriaged
Difficulté 2/5 1-3 heures Accessibilité débutants 64/100
dotnet/runtime#137126 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
bug debugger flaky small
Difficulté 2/5 1-3 heures Accessibilité débutants 77/100
Les mainteneurs répondent en général sous 1 jour