ci: validar tambien los PR que apuntan a otra rama feature - #23
Merged
Conversation
El harness SDD apila ramas: el PR de un ticket sale contra la rama del ticket anterior, no contra develop. Con el trigger limitado a [develop, main], de una pila de nueve PR solo se comprobaba el primero y los otros ocho llegaban a revisión sin que nadie hubiera pasado ktlint ni los tests sobre su árbol. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01E41idMYhfgBSz2EeuNUn5Q
hgarciaalberto
pushed a commit
that referenced
this pull request
Aug 19, 2026
…21) Cierra de una vez la pila 003→010, que estaba encadenada PR sobre PR. Sustituye a #14, #15, #16, #17, #18, #19 y #20. - 003 Apple Sign-In de producción en el demo y contrato correcto para Swift - 004 Cablea GitHub, Microsoft, Twitter, Facebook y teléfono en el demo (+ su spec) - 005 Revoca el token de Apple antes de borrar la cuenta - 006 Cierra también la sesión de GoogleSignIn al salir, en iOS - 007 Pasa los scopes de AppleSignInConfig al handler de iOS - 008 Fija el bundle id de iOS y alinea el magic link - 009 Sesión de teléfono completa y testeable en las dos plataformas - 010 Documenta lo que falta para construir el demo y deprecar código muerto Las ocho PRs intermedias apuntaban cada una a la rama de la anterior, y el CI solo disparaba en PRs contra develop/main, así que ninguna había pasado un check. Eso lo arregló #23. Esta pila se validó entera antes de entrar: ambos jobs en verde.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
El harness SDD apila ramas: el PR de un ticket sale contra la rama del ticket anterior, no contra
develop. Con el trigger limitado a[develop, main], de la pila de nueve PR que acaba de abrirse (#14 … #22) solo se comprueba el #14; los otros ocho llegan a revisión sin que nadie haya pasado ktlint ni los tests sobre su árbol concreto.Coste
Cada push a una rama de la pila pasa a lanzar también el job
apple, que corre enmacos-latesty factura a 10x. Si eso pesa, la alternativa es condicionar ese job a que el cambio toquecustom-login/src/iosMain/**oiosApp/**. Pero el de macOS es el único job que compila Kotlin/Native, así que dejarlo fuera de las ramas feature es perder justo la comprobación que más falta hace aquí.Nota
Esto no valida retroactivamente los ocho PR ya abiertos: el trigger actúa en el siguiente push de cada rama. Para cubrir el hueco ya se ha lanzado un
workflow_dispatchsobrefeature/010-ios-build-hygiene, que es la punta de la pila implementada y contiene la unión de todos los cambios de Kotlin de los tickets 003–010.🤖 Generated with Claude Code
https://claude.ai/code/session_01E41idMYhfgBSz2EeuNUn5Q