[workshop-sync] side-quest-11-05-event-triggers.md: PR reviewer references point to wrong step (15-conditional-logic.md instead of 14b-pr-review
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 78/100
- Tipo de issue
- Documentación
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- github-actions
- Área
- documentation
Línea de trabajo
Empieza en workshop/side-quest-11-05-event-triggers.md y encuentra cada enlace o etiqueta que apunte a 15-conditional-logic.md como PR Code Reviewer, además de la etiqueta 'Step 11c' y la tabla comparativa Step 7 vs Step 15. Revisa workshop/14b-pr-reviewer-workflow.md para ver el título real y su trigger pull_request y el safe output submit-pull-request-review, y workshop/README.md para la tabla del currículo. Está terminado cuando cada referencia apunte a 14b con el título correcto y la fila de la tabla coincida con el trigger y el safe output de 14b.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Workshop file reviewed
workshop/side-quest-11-05-event-triggers.md
Problem
This side quest repeatedly calls 15-conditional-logic.md the "PR Code Reviewer" step and labels it "Step 11c" / "Step 15":
...before you finish Build — PR Code Reviewer, then return to the main adventure.
This is a good fit when you want feedback tied to the current PR, like the PR Code Reviewer in Step 11c.
The Daily Repo Status workflow in Step 7 and the PR Code Reviewer in Step 15 use the same workflow format...
Return to the main adventure: Build — PR Code Reviewer.
But workshop/15-conditional-logic.md is actually titled "Make Your Workflow Smarter with Conditional Logic" — it adds an if: guard to the existing schedule-triggered daily-status workflow from Step 7. It has no pull_request trigger, no PR-review content, and no add-comment/create-issue safe-outputs block at all.
The actual PR reviewer workshop content lives at workshop/14b-pr-reviewer-workflow.md ("Build a PR Reviewer with an Agent and Skill"), confirmed by the curriculum table in workshop/README.md:
| 14b | [Build Your First Event-Driven Workflow: PR Auto-Reviewer](14b-pr-reviewer-workflow.md) |
| 15 | [Make Your Workflow Smarter with Conditional Logic](15-conditional-logic.md) |
There is also no "Step 11c" anywhere in the curriculum — the nearest step is 14b.
Additionally, the "Concrete example: Step 7 vs Step 15" table claims Step 15 uses add-comment as its safe output, but Step 15 doesn't define a trigger or safe-outputs change at all (it only adds an if: condition to Step 7's workflow) — this table row should reference Step 14b's actual trigger (pull_request: types: [ready_for_review]) and safe output (submit-pull-request-review, per workshop/14b-pr-reviewer-workflow.md), not Step 15.
Current correct behavior
- The PR Code Reviewer content is Step 14b (
14b-pr-reviewer-workflow.md), not Step 15 or "Step 11c". - Step 14b's workflow uses
on: pull_request: types: [ready_for_review]andsafe-outputs: submit-pull-request-review(notadd-comment).
Suggested fix
- Replace all links/labels pointing to
15-conditional-logic.mdas "PR Code Reviewer" with links to14b-pr-reviewer-workflow.md, using its real title "Build a PR Reviewer with an Agent and Skill" (or "PR Auto-Reviewer" per the README). - Change "Step 11c" to "Step 14b".
- Update the "Concrete example" comparison table to reference Step 14b's actual trigger/safe-output pair instead of Step 15's.
Generated by 🔍 Workshop Sync Check · copilot · auto · 183 AIC · ⌖ 6.38 AIC · ⊞ 9.2K · ◷
- expires on Oct 10, 2026, 5:02 AM UTC
- Lenguaje dominante
- JavaScript
- Estrellas
- 52
- Forks
- 26
- Merge medio
- 12 h 58 min
- PR fusionados (30 d)
- 15
Preparar el entorno
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Sin guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de githubnext/gh-aw-workshop
-
documentation
Dificultad 1/5 Menos de una hora Aptitud para principiantes 95/100
githubnext/gh-aw-workshop#4535 ·
Los mantenedores suelen responder en 1 día
-
agentic-workflows documentation workflow-editor
Dificultad 2/5 1-3 horas Aptitud para principiantes 63/100
githubnext/gh-aw-workshop#4533 ·
Los mantenedores suelen responder en 1 día
-
documentation
Dificultad 1/5 1-3 horas Aptitud para principiantes 88/100
githubnext/gh-aw-workshop#4532 ·
Los mantenedores suelen responder en 1 día
-
documentation
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
githubnext/gh-aw-workshop#4531 ·
Los mantenedores suelen responder en 1 día
-
curriculum documentation quality
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
githubnext/gh-aw-workshop#4528 ·
Los mantenedores suelen responder en 1 día
Todos los issues de githubnext/gh-aw-workshop
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
-
[quality] Nothing keeps docs/skills/manifest.md in sync with docs/skills/*.mdPosiblemente ocupada @hivecommons-hive la tomó hoy. Abiertoagent/quality hive/covered-by-pr hive/hosted-available-lke648397-260827-5n31 quality testing
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
agent:blocked agent:ready
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
turbot/guardrails-fn#46 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
needs verification
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
Los mantenedores suelen responder en 6 días