preview.yml default globs ./packages/** and ./examples/** recurse into node_modules
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 68/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Tranquilo
- Stack tecnológico
- github-actions
- Área
- ci-cd
Línea de trabajo
Comienza en preview.yml inspeccionando los globs predeterminados de publish y template y cómo se pasan a pkg-pr-new. Usa la ejecución enlazada de clink preview como reproducción y verifica después que el paso de publish ya no recorra node_modules anidados ni falle con archivos package.json sin nombre.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem
preview.yml defaults its inputs to recursive globs:
publish:
default: "./packages/**"
template:
default: "./examples/**"
and passes them to pkg-pr-new:
pnpx pkg-pr-new publish --pnpm "$PUBLISH_GLOB" --template "$TEMPLATE_GLOB"
The ** matches recursively, including into each package's node_modules. pkg-pr-new then walks nested dependency package.json files and aborts on any that lack a name field — which is common for internal stub manifests such as vite's:
[error] "name" field in .../packages/clink/node_modules/vite/types/package.json should be defined
##[error]Process completed with exit code 1.
(Preceded by many Skipping .../node_modules/.../*.ts: not a directory or .tgz/.tar.gz file lines as it scans the dependency tree.)
Impact
Any repo whose packages depend on something that ships a nameless nested package.json (vite is a frequent one). The preview job fails at the publish step even though install and build succeed. Latent until the build step actually completes and pkg-pr-new runs.
Repro
- Concrete instance: bombshell-dev/clink#43 — the
previewrun that reached the "Publish Preview" step:
https://github.com/bombshell-dev/clink/actions/runs/30575154965
- Lenguaje dominante
- TypeScript
- Estrellas
- 4
- Forks
- 1
- 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 bombshell-dev/automation
-
Dificultad 4/5 3-5 días Aptitud para principiantes 55/100
bombshell-dev/automation#33 ·
Todos los issues de bombshell-dev/automation
Issues similares
-
Add: Digi World (Romania) SDAbiertocheck:passed streams:add
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Fission-AI/OpenSpec#1986 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
nestjs/docs.nestjs.com#3554 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
publicodes/publicodes#868 ·
-
namespace operations
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
EclipseFdn/open-vsx.org#13488 ·
Los mantenedores suelen responder en 2 días