insertSpaces formatting option doesn't always work
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Stack tecnológico
- typescript
- Área
- tooling
Línea de trabajo
Empieza ejecutando la reproducción proporcionada con ts-node y siguiendo los puntos de entrada format y applyEdits. Añade una prueba de regresión que cubra un tabulador dentro de un valor entre comillas junto con un tabulador de indentación, y verifica que el formateo elimine únicamente el tabulador que está fuera de las comillas.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Steps to reproduce. Run the following program using ts-node or similar.
import { format, applyEdits } from 'jsonc-parser';
const JSONString = `{"hello": \t"\tworld"}`;
const edit = format(JSONString, undefined, { insertSpaces: true});
const res = applyEdits(JSONString, edit);
console.log(res);
We would expect the following output
{
"hello": " world"
}
But we actually get
{
"hello": " world"
}
Interestingly, if we remove the tab inside the quotes, the tab outside is removed as expected. This seems like an error since formatting should only care about tabs outside quotes.
- Lenguaje dominante
- TypeScript
- Estrellas
- 759
- Forks
- 66
- Merge medio
- 5 d 10 h
- PR fusionados (30 d)
- 7
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
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 microsoft/node-jsonc-parser
-
Dificultad 3/5 1-2 días Aptitud para principiantes 72/100
microsoft/node-jsonc-parser#125 · 1 comentario ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
microsoft/node-jsonc-parser#105 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
microsoft/node-jsonc-parser#103 · 2 comentarios · 8 reacciones ·
-
Provide usage examples Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 48/100
microsoft/node-jsonc-parser#97 · 3 reacciones ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
microsoft/node-jsonc-parser#95 · 1 reacción ·
Todos los issues de microsoft/node-jsonc-parser
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
receptron/mulmoterminal#2264 ·
-
documentation
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
components-web-app/docs#96 ·
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
simonsobs/tileviewer#114 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100