make lint fails on shellcheck style and info warnings
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 84/100
Línea de trabajo
Ejecuta make lint para reproducir el fallo de ShellCheck y, a continuación, inspecciona las líneas indicadas en gh-md-toc. Corrige las advertencias de estilo SC2006 y SC2181 y gestiona las expresiones awk intencionales según lo indicado. La tarea estará terminada cuando make lint finalice correctamente con ShellCheck 0.11.0.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Steps
make lint
Actual
shellcheck -e SC2008 gh-md-toc exits with code 1 on master (c386c7b, shellcheck 0.11.0):
- SC2006 (style): legacy backticks instead of
$(...)- lines 170, 187, 246, 305-308, 371 - SC2181 (style): exit code checked indirectly with
$?- line 89 - SC2016 (info): expressions in single quotes - lines 249, 258
Expected
make lint passes.
Notes
- CI (
ci.yml) runs onlymake test, so the failure is not visible there. - Lines 249 and 258 start the awk scripts, which are single-quoted on purpose so that
$0reaches awk unexpanded. SC2016 there likely needs a disable directive rather
than a code change.
- Lenguaje dominante
- Shell
- Estrellas
- 3.3k
- Forks
- 2.7k
- Merge medio
- 2 min
- PR fusionados (30 d)
- 1
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Sin 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 ekalinin/github-markdown-toc
-
documentation
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
ekalinin/github-markdown-toc#165 ·
-
bug
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
ekalinin/github-markdown-toc#166 ·
-
[Feature] `--no-code-tag-removal` flag allowing to make removal of backticks in TOC entries optionalAbiertoenhancement
Dificultad 3/5 1-2 días Aptitud para principiantes 58/100
ekalinin/github-markdown-toc#153 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
ekalinin/github-markdown-toc#152 · 1 reacción ·
-
bug
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
ekalinin/github-markdown-toc#141 ·
Todos los issues de ekalinin/github-markdown-toc
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
YosysHQ/oss-cad-suite-build#216 ·
-
limine: new version 12.9.1Abiertoout-of-date
Dificultad 1/5 Menos de una hora Aptitud para principiantes 82/100
CachyOS/CachyOS-PKGBUILDS#1917 ·
Los mantenedores suelen responder en 1 día
-
package-update
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
bug needs triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
EmbarkStudios/cargo-about#323 ·