Record frame: remaining review nits (PR #147)
Mantenedores costumam responder em até 1 dia
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 5/5
- Tempo estimado
- Mais de uma semana
- Facilidade para iniciantes
- 35/100
Direção de pesquisa
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.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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.
- Linguagem predominante
- C#
- Estrelas
- 0
- Forks
- 0
- Merge médio
- 1h 43min
- PRs com merge (30d)
- 43
Preparar o ambiente
Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de sillsdev/SIL.TextEditing
-
Playground selection bar: comes back after Apply or scrolling while focus is in the Markdown paneAberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
sillsdev/SIL.TextEditing#175 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
sillsdev/SIL.TextEditing#167 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
sillsdev/SIL.TextEditing#164 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
sillsdev/SIL.TextEditing#137 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 1/5 1-3 horas Facilidade para iniciantes 82/100
sillsdev/SIL.TextEditing#133 ·
Mantenedores costumam responder em até 1 dia
Todas as issues de sillsdev/SIL.TextEditing
Issues semelhantes
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 74/100
juandresrodca/AdminForge#54 ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
Mantenedores costumam responder em até 1 dia
-
要トリアージ 辞書
Dificuldade 1/5 1-3 horas Facilidade para iniciantes 86/100
yksr-melt/Meltype#1625 · 1 reação ·
Mantenedores costumam responder em até 1 dia
-
Docs typoAberta
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 78/100
MonoGame/docs.monogame.github.io#328 · 1 comentário ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100
space-wizards/RobustToolbox#7197 ·
Mantenedores costumam responder em até 1 dia