Skip to content

Evitar deploys simultáneos en el CI y subir el timeout de tests - #83

Merged
Winbit-Finance merged 2 commits into
mainfrom
fix/ci-concurrency
Nov 5, 2026
Merged

Winbit-Finance merged 2 commits into
mainfrom
fix/ci-concurrency

Conversation

@Winbit-Finance

Copy link
Copy Markdown
Collaborator

Summary

  • ci.yml: agrega concurrency (group: ci-${{ github.ref }}, cancel-in-progress: true). Si hay dos corridas sobre la misma rama, la nueva cancela a la vieja. Evita que un deploy viejo termine después de uno nuevo y lo pise: pasó al mergear Parametrizar el mail de soporte con VITE_SUPPORT_EMAIL #81 y Redirigir las URLs viejas de Firebase a app.winbit.com.ar #82 seguidos, y producción quedó sin la redirección hasta re-ejecutar el deploy.
  • vite.config.js: testTimeout: 15000 (antes 5 s por defecto). DepositForm, WithdrawalForm y App daban timeout de forma intermitente con la suite completa en máquinas lentas, y bloqueaban el hook de pre-push; aisladas pasan siempre.

Notas

  • Con la concurrencia, si se hacen dos merges seguidos solo completa el CI del último, que ya incluye al primero.
  • No cambia código de la app.

Test plan

  • prettier --check sobre ci.yml y vite.config.js
  • Pre-push (lint, format, test:ci) en verde con el nuevo timeout
  • Tras el merge: hacer dos merges seguidos y confirmar que la corrida vieja queda cancelada y el deploy final es el del último commit

@Winbit-Finance
Winbit-Finance merged commit 41ee4d3 into main Nov 5, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant