Repository navigation
chore(config): plantillas de issue tarea_actividad y rfc_change (Segundo Parcial) - #10
Merged
Merged
Conversation
This was referenced Oct 30, 2026
The-Fabry
approved these changes
Oct 30, 2026
The-Fabry
left a comment
Collaborator
There was a problem hiding this comment.
✅ Aprobado (LGTM)
He revisado los cambios y todo se ve excelente. Buen trabajo con la configuración inicial del repositorio.
Puntos destacados:
- Ubicación correcta: Las plantillas
tarea_actividad.mdyrfc_change.mdestán bien estructuradas dentro de.github/ISSUE_TEMPLATE/. - Claridad: La descripción del PR detalla perfectamente el propósito de cada archivo, incluyendo el ciclo de vida CCB para el RFC.
- Buenas prácticas: Se respeta el flujo de trabajo (rama desde
develop), el formato de commit (chore(config)) y se mantiene limpio sin código de aplicación.
Validación del Checklist:
- Rama creada desde develop
- Commits con formato chore(config): ...
- Sin código de aplicación, solo configuración
- Pendiente la segunda aprobación requerida para hacer el merge.
¡Todo listo por mi parte! Gracias.
levi06516-ui
left a comment
Collaborator
There was a problem hiding this comment.
3. Revisión del Pull Request (Aprobación)
¡Excelente trabajo con la configuración inicial del repositorio! He revisado los cambios a detalle y todo quedó perfecto.
Puntos destacados:
- Organización impecable: Las plantillas
tarea_actividad.mdyrfc_change.mdse ubicaron correctamente dentro de.github/ISSUE_TEMPLATE/. - Claridad total: La descripción del PR explica de forma clara el propósito de cada plantilla, incluyendo la integración del ciclo de vida CCB para los RFC.
- Rigor técnico: Se siguió el flujo de trabajo al pie de la letra (rama derivada de
develop), manteniendo un historial limpio con el formato de commitchore(config)y sin incluir código fuente innecesario.
Verificación del Checklist:
- Rama creada a partir de
develop - Commits siguiendo el estándar
chore(config): ... - PR enfocado exclusivamente en archivos de configuración
- Queda pendiente únicamente la segunda aprobación para proceder con el merge
¡Todo aprobado por mi parte! Listo para avanzar.
levi06516-ui
approved these changes
Oct 31, 2026
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
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.
Cambios incluidos
Closes (ningún issue previo, es configuración inicial del repositorio).
Checklist