Have formatter insert newlines around multi-line expressions
Mantenedores costumam responder em até 1 dia
Avaliação
- Dificuldade
- 4/5
- Tempo estimado
- 3-5 dias
- Facilidade para iniciantes
- 25/100
- Tipo de issue
- Funcionalidade
- Clareza
- Razoavelmente clara
- Status de atividade
- Estagnada
- Stack de tecnologia
- ocaml
- Domínio
- tooling
Direção de pesquisa
Start by reviewing the linked draft pull request #8961 to see the formatter approach and the files or tests it touches. Then trace how the formatter handles whitespace around multi-line expressions and compare the output with the example in this issue. Done when formatting adds consistent whitespace around multi-line expressions and the relevant formatter tests pass.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
module Details = {
@react.component
let make = (~input as {filePath}: input) => React.string(filePath)
}
module Body = {
@react.component
let make = (~input as {filePath, oldString, newString, replaceAll}: input) => <>
<ToolView.Field label="file" value=filePath />
<ToolView.Text
label={replaceAll->Option.getOr(false) ? "replace all" : "replace"} text=oldString
/>
{newString == ""
? <ToolView.Field label="with" value="(nothing)" />
: <ToolView.Text label="with" text=newString />}
</>
}
module Output = Tool.TextResult.Output
let failed = Tool.TextResult.failed
It seems that some AIs love writing very dense code, and it makes things rather unreadable. It would be amazing if we could print in a way that adds white-spacing / makes the formatting consistent. Just like we remove double white-spacing in the formatter
I have a draft PR (vibed) here: https://github.com/rescript-lang/rescript/pull/8961
- Linguagem predominante
- OCaml
- Estrelas
- 7.5k
- Forks
- 483
- Merge médio
- 19h 35min
- PRs com merge (30d)
- 208
Preparar o ambiente
Inicia o contêiner de desenvolvimento do projeto no navegador, com a sua própria conta do GitHub.
- Sem Dockerfile nem arquivo Docker Compose
- Tem um modelo de pull request
- Ler o guia de contribuição
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 rescript-lang/rescript
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
rescript-lang/rescript#9037 ·
Mantenedores costumam responder em até 1 dia
-
Per-constructor @unboxed is not compared across interfaces or coercionsTalvez já em andamento @cknitt assumiu hoje. Aberta
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 25/100
rescript-lang/rescript#9038 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 35/100
rescript-lang/rescript#9006 · 1 reação ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 70/100
rescript-lang/rescript#8997 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 68/100
rescript-lang/rescript#8994 ·
Mantenedores costumam responder em até 1 dia
Todas as issues de rescript-lang/rescript
Issues semelhantes
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
eugenioenko/ttt#1206 ·
Mantenedores costumam responder em até 1 dia
-
correctness good first issue provider
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 88/100
Mantenedores costumam responder em até 1 dia
-
agent prompt
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
fritzprix/libr-agent#2328 ·
Mantenedores costumam responder em até 1 dia
-
area/tools bug session-analysis
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
Mantenedores costumam responder em até 1 dia
-
bug ready-for-pr
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 71/100
kunchenguid/no-mistakes#1615 · 1 comentário ·
Mantenedores costumam responder em até 1 dia