[Blocker] Add a PR CI workflow and make tests gate the deploy
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 65/100
Línea de trabajo
Empieza leyendo .github/workflows/azure-app-service.yml alrededor de las líneas 91-92 e inspecciona los destinos de compilación de LearnStack.sln y LearnStack.csproj. Añade .github/workflows/ci.yml usando restore, solution build y dotnet test para pull requests y pushes, y después actualiza el workflow de deploy. El trabajo está terminado cuando las comprobaciones PR requeridas y las pruebas de deploy fallen visiblemente si falla una prueba, y el resumen del workflow muestre los resultados.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Tier 1 — release blocker
Problem
Three separate gaps:
- The deploy workflow runs tests with
continue-on-error: true, so a failing test never blocks a production deploy. - It runs
dotnet test --no-buildat solution level after building onlyLearnStack.csproj, so the test project is likely never built and the tests probably do not execute at all — the swallowed error hides this. - There is no PR build workflow at all: the only workflow triggers on push to
mainand deploys straight to production.
Evidence
.github/workflows/azure-app-service.yml:91-92.github/workflows/contains onlyazure-app-service.yml
Proposed fix
- Add
.github/workflows/ci.yml: restore, buildLearnStack.sln,dotnet testonpull_requestandpush. - Mark it a required status check on
main. - In the deploy workflow, build the solution and drop
continue-on-error.
Acceptance criteria
- A PR with a failing test cannot be merged
- A failing test blocks the deploy job
- Test results are visible in the workflow summary
- Lenguaje dominante
- HTML
- Estrellas
- 10
- Forks
- 0
- Merge medio
- 17 min
- PR fusionados (30 d)
- 9
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin 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 kasuken/LearnStack
-
enhancement
Dificultad 1/5 Menos de una hora Aptitud para principiantes 91/100
kasuken/LearnStack#59 ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
kasuken/LearnStack#72 ·
Los mantenedores suelen responder en 1 día
-
Feature: admin dashboardAbiertoenhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 30/100
kasuken/LearnStack#71 ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
kasuken/LearnStack#70 ·
Los mantenedores suelen responder en 1 día
-
Feature: public profile pagesAbiertoenhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 45/100
kasuken/LearnStack#69 ·
Los mantenedores suelen responder en 1 día
Todos los issues de kasuken/LearnStack
Issues similares
-
[BUG] deploy.restart_policy without condition drops the restart policyPosiblemente ocupada @akshat-lakhera la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
docker/compose#14383 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
needs-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
melodic-software/claude-code-plugins#9619 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
Los mantenedores suelen responder en 1 día
-
body-parser 2.3.0 in the published ai-dial-chat-bff image is vulnerable (CVE-2026-111181, HIGH)Posiblemente ocupada @illya-havsiyevych la tomó hoy. Abiertobug
Dificultad 1/5 1-3 horas Aptitud para principiantes 87/100
epam/ai-dial-chat#9566 ·
Los mantenedores suelen responder en 1 día
-
code-maintenance
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
Los mantenedores suelen responder en 1 día