Have formatter insert newlines around multi-line expressions
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 25/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- ocaml
- Área
- tooling
Línea de trabajo
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.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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
- Lenguaje dominante
- OCaml
- Estrellas
- 7.5k
- Forks
- 483
- Merge medio
- 20 h
- PR fusionados (30 d)
- 219
Preparar el entorno
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de rescript-lang/rescript
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
rescript-lang/rescript#9037 ·
Los mantenedores suelen responder en 1 día
-
RFC: Temporal API bindingsAbierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 12/100
rescript-lang/rescript#9044 · 2 reacciones ·
Los mantenedores suelen responder en 1 día
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
rescript-lang/rescript#9006 · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
Dificultad 3/5 1-2 días Aptitud para principiantes 70/100
rescript-lang/rescript#8997 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
rescript-lang/rescript#8994 ·
Los mantenedores suelen responder en 1 día
Todos los issues de rescript-lang/rescript
Issues similares
-
follow-up
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
[BUG] jackson3 JacksonJsonpMapper(ObjectMapper) fails to deserialize `_source` with "Trailing token" errorPosiblemente ocupada @junyeokk la tomó hoy. Abiertobug untriaged
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
opensearch-project/opensearch-java#2236 ·
Los mantenedores suelen responder en 1 día
-
Syntax error with CONST-sAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
FinanceFlash/unvibecode#282 ·