CI does not run on stacked PRs (pull_request trigger restricted to main)

Abierto Apto para principiantes
#63 1 comentario 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
1/5
Tiempo estimado
Menos de una hora
Aptitud para principiantes
76/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Tranquilo
Stack tecnológico
github-actions
Área
ci-cd

Línea de trabajo

Abre .github/workflows/ci.yml e inspecciona el disparador pull_request y su filtro de ramas. Compara el comportamiento del workflow para un pull request dirigido a main con uno dirigido a otra rama y, después, verifica que los PRs apilados reciban el build, las pruebas de Zig, las pruebas de JavaScript y las comprobaciones de lint. Se considera completado cuando los pull requests contra cualquier rama base activan CI, mientras que los pushes siguen limitados a main.

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

Descripción

Type: Maintenance

Context

Noted while reviewing #56, which targets the fix/napi-out-parameters-v2 branch (stacked on #57). No CI checks ran on it because .github/workflows/ci.yml triggers only on:

on:
  pull_request:
    branches: [main]

Any PR whose base is another feature branch gets zero checks (build, zig tests, JS tests, lint), so stacked PRs can only be validated locally until the base PR merges or the PR is retargeted.

Proposal

Drop the branch filter on the pull_request trigger (or widen it) so CI runs for PRs against any base branch:

on:
  pull_request:
  push:
    branches: [main]

Ref: .github/workflows/ci.yml

Lenguaje dominante
Zig
Estrellas
4
Forks
4
Sin PR fusionados en 30 d

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

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 ChainSafe/zapi

Todos los issues de ChainSafe/zapi

Issues similares

Más issues de DevOps

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.