Conversation
|
|
✅ Deploy Preview for secorto-astro ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
SeCOrTo web
|
||||||||||||||||||||||||||||
| Project |
SeCOrTo web
|
| Branch Review |
pull/87/head
|
| Run status |
|
| Run duration | 01m 34s |
| Commit |
|
| Committer | Sergio Orozco |
| View all properties for this run ↗︎ | |
| Test results | |
|---|---|
Failures
|
0
|
Flaky
|
0
|
Pending
|
0
|
Skipped
|
0
|
Passing
|
6
|
| View all changes introduced in this branch ↗︎ | |
There was a problem hiding this comment.
Pull request overview
This pull request creates retrospective Architecture Decision Records (ADRs) that document the complete evolution of the secorto_web project through three major platform migrations spanning from 2016 to present. It addresses issue #85 by reconstructing historical decisions from git history to provide comprehensive documentation of past architectural choices and their outcomes.
Changes:
- Adds a new "ADRs retrospectivos" section to the ADR index, clearly distinguishing historical decisions from current project ADRs
- Documents the complete lifecycle of three platforms: Jekyll (2016-2021), Gatsby (2021-2024), and Astro (2024-present)
- Establishes the
Rprefix naming convention for retrospective ADRs to maintain chronological clarity
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| docs/adr/README.md | Adds retrospective ADRs section to the index with a new table and updates conventions to document the R prefix naming pattern |
| docs/adr/R01-fundacion-sitio-jekyll.md | Documents the initial Jekyll-based site (2016-2021), covering 423 commits across 6 phases from Gulp+Bootstrap to Minimal Mistakes theme and Netlify CMS |
| docs/adr/R02-migracion-jekyll-a-gatsby.md | Documents the migration to Gatsby (2021-2023), introducing React components, MDX, Theme UI, Jest snapshot testing, and Cypress E2E tests |
| docs/adr/R03-migracion-gatsby-a-astro.md | Documents the current Astro migration (2024-present), detailing the transition from 34 Gatsby plugins to 3 Astro integrations, implementation of i18n, and adoption of Playwright+Vitest testing |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
| | [R01](R01-fundacion-sitio-jekyll.md) | Fundación del sitio personal con Jekyll | Reemplazada → R02 | 2016-04 | | ||
| | [R02](R02-migracion-jekyll-a-gatsby.md) | Migración de Jekyll a Gatsby | Reemplazada → R03 | 2021-03 | |
There was a problem hiding this comment.
En el índice de ADRs retrospectivos, el estado “Reemplazada → R02/R03” no incluye enlace al ADR sustituto. Esto dificulta la navegación y además contradice la convención indicada más abajo (“Reemplazada con referencia al ADR que la sustituye”). Conviene enlazar el ADR de reemplazo en la columna Estado (p. ej. “Reemplazada → R02”).
| | [R01](R01-fundacion-sitio-jekyll.md) | Fundación del sitio personal con Jekyll | Reemplazada → R02 | 2016-04 | | |
| | [R02](R02-migracion-jekyll-a-gatsby.md) | Migración de Jekyll a Gatsby | Reemplazada → R03 | 2021-03 | | |
| | [R01](R01-fundacion-sitio-jekyll.md) | Fundación del sitio personal con Jekyll | Reemplazada → [R02](R02-migracion-jekyll-a-gatsby.md) | 2016-04 | | |
| | [R02](R02-migracion-jekyll-a-gatsby.md) | Migración de Jekyll a Gatsby | Reemplazada → [R03](R03-migracion-gatsby-a-astro.md) | 2021-03 | |
| Migrar a **Gatsby** (v5) con React 18, Theme UI para estilos, MDX para | ||
| contenido, y Jest + Cypress como frameworks de testing. | ||
|
|
||
| ### Stack adoptado | ||
|
|
||
| | Capa | Tecnología | Reemplaza a | | ||
| |---|---|---| | ||
| | SSG | Gatsby 5 | Jekyll | |
There was a problem hiding this comment.
Hay una inconsistencia interna de versión: en “Decisión” se indica “Gatsby (v5)”, pero en “Métricas del repositorio” se afirma “Versión final: 4.2.0 (package.json)”. Ajustar uno de los dos para que el ADR sea coherente (o explicar el motivo si el repo empezó en v5 pero terminó en v4.x).
| Migrar a **Gatsby** (v5) con React 18, Theme UI para estilos, MDX para | |
| contenido, y Jest + Cypress como frameworks de testing. | |
| ### Stack adoptado | |
| | Capa | Tecnología | Reemplaza a | | |
| |---|---|---| | |
| | SSG | Gatsby 5 | Jekyll | | |
| Migrar a **Gatsby 4** (versión final 4.2.0) con React 18, Theme UI para estilos, | |
| contenido en MDX, y Jest + Cypress como frameworks de testing. | |
| ### Stack adoptado | |
| | Capa | Tecnología | Reemplaza a | | |
| |---|---|---| | |
| | SSG | Gatsby 4 (4.2.0) | Jekyll | |
| # ADR R01: Fundación del sitio personal con Jekyll | ||
|
|
||
| > **Estado:** Reemplazada → [R02](R02-migracion-jekyll-a-gatsby.md) | ||
| > **Fecha original:** 2016-04 (reconstruido retrospectivamente 2026-02) | ||
| > **Categoría:** Plataforma / SSG / Quality | ||
| > **Repositorio:** `secorto.com_jekyll` (423 commits, 2016–2023) | ||
|
|
There was a problem hiding this comment.
Este ADR se marca como “reconstruido retrospectivamente”, pero no incluye una sección explícita de retrospectiva con los campos que el issue #85 pide para los ADRs retrospectivos (p. ej. “Decisión vigente (sí/no/parcial)”, resumen breve, si requiere actualización, responsable/fecha límite y acciones). Si este PR cierra #85, conviene añadir esa sección al final (o en el frontmatter/encabezado) para que quede verificable y accionable.
| # ADR R02: Migración de Jekyll a Gatsby | ||
|
|
||
| > **Estado:** Reemplazada → [R03](R03-migracion-gatsby-a-astro.md) | ||
| > **Fecha original:** 2021-03 (reconstruido retrospectivamente 2026-02) | ||
| > **Categoría:** Plataforma / SSG / Testing / Quality | ||
| > **Repositorio:** `web2021` (111 commits, 2021–2023) | ||
|
|
There was a problem hiding this comment.
Este ADR se marca como “reconstruido retrospectivamente”, pero no incluye una sección explícita de retrospectiva con los campos que el issue #85 pide para los ADRs retrospectivos (p. ej. “Decisión vigente (sí/no/parcial)”, resumen breve, si requiere actualización, responsable/fecha límite y acciones). Si este PR cierra #85, conviene añadir esa sección al final (o en el frontmatter/encabezado) para que quede verificable y accionable.
| # ADR R03: Migración de Gatsby a Astro | ||
|
|
||
| > **Estado:** Aceptada | ||
| > **Fecha original:** 2024-05 (reconstruido retrospectivamente 2026-02) | ||
| > **Categoría:** Plataforma / SSG / Testing / i18n | ||
| > **Repositorio:** `secorto_web` (502+ commits, 2024–presente) | ||
|
|
There was a problem hiding this comment.
Este ADR se marca como “reconstruido retrospectivamente”, pero no incluye una sección explícita de retrospectiva con los campos que el issue #85 pide para los ADRs retrospectivos (p. ej. “Decisión vigente (sí/no/parcial)”, resumen breve, si requiere actualización, responsable/fecha límite y acciones). Si este PR cierra #85, conviene añadir esa sección al final (o en el frontmatter/encabezado) para que quede verificable y accionable.
Pull request
Closes #85
Observaciones
Se crean ADR retrospectivos y se documenta cada decisión