search

scartasegna / hado

Terraform for creating an ECS cluster and deploy a container on each merge
0 stars 0 forks source link

Trigger on PR Merge #11

Open pparodi-chelagarto opened 4 weeks ago

pparodi-chelagarto commented 4 weeks ago

https://github.com/scartasegna/hado/blob/6159b313f3b95135a1ac3408b111014a16350094/.github/workflows/actions.yml#L3

Evaluar poner: ` on: push: branches:

scartasegna commented 4 weeks ago

Si agrego on: push: branches: - master se evalua por cada push al branch main se inicia una ejecución, cierto? De igual forma el if en la definición del debería frenarlo (to be tested) En la descripción decia que solo debía ejecutar al mergear a main y por eso está definido que solo ejecute en close de los pull request en el branch main. El if en el job es solo para verificar que debe ejecutar en el caso que PR sea cerrado y mergedo

scartasegna commented 4 weeks ago

Deje que se ejecute en los push a main, porque no encontre la forma de identificar que el push se esta ejecutando desde un merge en los eventos de push