ER: Blank PRs get commented on for formatting issues too

Abierto Apto para principiantes
  • ¿Algo no está bien?
#8,871 1 comentario 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
67/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
github-actions, javascript
Área
ci-cd, tooling

Línea de trabajo

Lee ~/github-actions/trigger-pr/check-linked-issue.js, especialmente la línea 105, donde el issue identifica un fallo cuando pr.body está en blanco. Empieza por examinar cómo la acción gestiona los cuerpos de las solicitudes de incorporación de cambios y revisa el informe del fallo enlazado en PR #8815; usa las guías de pruebas de GitHub Actions proporcionadas para ejecutarla en un entorno personal. Se considera terminado cuando la acción gestiona un cuerpo de PR en blanco y deja el comentario esperado.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Complexity: Small ER Feature: Refactor GHA Issue Making: Level 2 ready for merge team role: back end/devOps size: 0.25pt
Emergent Requirement - Problem

We should make sure that check-linked-issue.js can handle blank PRs as well. That code currently helps to check PRs for proper formatting to make sure an issue is linked, but it fails if the PR it is checking is blank.

Issue you discovered this emergent requirement in
  • #8815
Date discovered

10-24-2026

Did you have to do something temporarily
  • YES
  • NO
Who was involved

@castillios
@daras-cu

What happens if this is not addressed

If a blank PR get published, the PR to check for a a properly linked issue does not leave any comment on the PR to notify the author of an issue.

Resources
Recommended Action Items
  • Make a new issue
  • Discuss with team
  • Let a Team Lead know
Potential solutions [draft]

Issue:

  • Complexity: Small (While the suggested change here only requires changing one line of code, I gave it the Complexity: Small label because it requires testing GitHub Actions.)
  • Relationships: None
  • Size: 0.25pt
  • Fix:
    in ~/github-actions/trigger-pr/check-linked-issue.js (line 105), change:
const match = pr.body.match(regex);

... to:

const match = (pr.body ?? '').match(regex);
Lenguaje dominante
JavaScript
Estrellas
365
Forks
872
Merge medio
8 d 19 h
PR fusionados (30 d)
2

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de hackforla/website

Todos los issues de hackforla/website

Issues similares

Más issues de JavaScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.