Remove empty lines in some cases
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 30/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Necesita aclaración
- Estado de actividad
- Estancado
- Área
- tooling
Línea de trabajo
No se nombra ningún archivo, prueba ni punto de entrada. Empieza por localizar el manejo existente de líneas en blanco del formateador; después, aclara e implementa el comportamiento propuesto para listas de sentencias, declaraciones de rutinas y bloques de type/const/var/threadvar; se considera terminado cuando las reglas acordadas estén implementadas y probadas.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
While we let users keep at most one blank line to improve readability of code, there are some cases where we could be a bit more assertive and remove (or add) a blank line.
Here are some initial thoughts for how a rule like this could work. I'm interested in hearing other opinions on the matter.
statement lists should not start or end with a blank line
begin
A;
end;
try
A;
finally
B;
end;
routine declarations should follow and precede a blank line
procedure A;
begin
end;
procedure B;
begin
end;
// comments are
// still 'attached'
[SoAreAttributes]
procedure C;
begin
end;
// trailing comments are 'detached'
type, const, var, threadvar blocks should not start with a blank line
type
A = B;
C = D;
const
A = '';
B = 0;
- Lenguaje dominante
- Rust
- Estrellas
- 108
- Forks
- 20
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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 integrated-application-development/pasfmt
-
enhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 30/100
-
enhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
-
bug
Dificultad 3/5 1-2 días Aptitud para principiantes 48/100
-
bug
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
integrated-application-development/pasfmt#256 · 1 comentario ·
-
bug
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
Todos los issues de integrated-application-development/pasfmt
Issues similares
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 70/100
OpenZeppelin/openzeppelin-relayer#967 ·
Los mantenedores suelen responder en 1 día
-
DecisionOutcome::label() / n_dosed(): engine-side mapping so downstream needs no wildcard armAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
src/runs.rs:1987: a missing --source-run-id now answers 400 "Source run ... does not exist", not 404Abiertonightly-audit
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
antithesishq/snouty#602 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
libnativeapi/nativeapi#21 ·