testing: Add support for runtime environment variables to GitHub Workflows
Los mantenedores suelen responder en 3 días
@grayside ya está trabajando en esto.
Desde el 12/9/2023.
Evaluación
Este issue todavía no se ha evaluado.
Descripción
GitHub Workflows is the preferred test driver for tests in this repository, but the current workflow implementation has several limitations. One of them is the ability to set runtime environment variables that are specific to a Workflow.
Approach
Options:
- Hard code a central map of environment variables, and refactor code to ensure clean namespacing
- Create a mechanism to pass environment variables to our reusable test workflow
Option 2 is the most versatile with the least need to refactor existing code.
Option 1 has the further downside of placing test-specific parameters in a central file. I'm not sure if any changes to test.yml are triggering wide-scale test execution currently, but with this approach we'd likely need to trigger more tests from any change, which would include adding environment variables for unrelated tests.
@pattishin has asked that as part of the PR fixing this issue, we look for existing hard-coded values and shift them to environment variable configuration.
This issue is a blocker on implementation of GHA-based test workflows for a number of samples, with #3149 as the driver for research today.
- Lenguaje dominante
- JavaScript
- Estrellas
- 3k
- Forks
- 2k
- Merge medio
- 3 d 23 h
- PR fusionados (30 d)
- 30
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la 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 GoogleCloudPlatform/nodejs-docs-samples
-
eslintrc rules may be invalidPosiblemente ocupada @yuhuayang-google la tomó hace 12 días. Abiertopriority: p2 samples triage me type: bug
Dificultad 1/5 Menos de una hora Aptitud para principiantes 48/100
GoogleCloudPlatform/nodejs-docs-samples#4174 · 2 comentarios ·
Los mantenedores suelen responder en 3 días
-
priority: p2 samples type: bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 55/100
GoogleCloudPlatform/nodejs-docs-samples#4144 ·
Los mantenedores suelen responder en 3 días
-
priority: p2 samples triage me type: bug
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
GoogleCloudPlatform/nodejs-docs-samples#3859 ·
Los mantenedores suelen responder en 3 días
-
samples
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
GoogleCloudPlatform/nodejs-docs-samples#3601 · 1 reacción ·
Los mantenedores suelen responder en 3 días
-
priority: p3 samples triage me type: feature request
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
GoogleCloudPlatform/nodejs-docs-samples#3578 · 13 comentarios · 1 reacción ·
Los mantenedores suelen responder en 3 días
Todos los issues de GoogleCloudPlatform/nodejs-docs-samples
Issues similares
-
needs-website-changes
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
[Feature] 已有证书托管支持拖拽上传Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 63/100
-
[Bug]: [A11y] Missing accessible name on icon-only standalone links (WCAG 2.1 SC 2.4.4, 4.1.2)Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
jaegertracing/jaeger-ui#4596 ·
Los mantenedores suelen responder en 1 día
-
Add: CNC Region Calima [CO]Abiertochannels:add check:passed
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
Los mantenedores suelen responder en 9 días