Hacktoberfest 2026：メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

[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

クローズ 初心者向け
  • 何か問題がありますか？
#4,366 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

まだ誰も着手していません。

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
78/100
issue の種類
ドキュメント
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
github-actions
領域
documentation

調査の方向性

workshop/side-quest-11-05-event-triggers.md から始めて、PR Code Reviewer として 15-conditional-logic.md を指しているすべてのリンクとラベル、さらに 'Step 11c' のラベルと Step 7 vs Step 15 の比較表を見つけてください。workshop/14b-pr-reviewer-workflow.md では実際のタイトル、pull_request トリガー、submit-pull-request-review の safe output を確認し、workshop/README.md ではカリキュラムの表を確認してください。すべての参照が正しいタイトルで 14b を指しており、表の行が 14b のトリガーと safe output に一致していれば完了です。

索引モデルが issue の本文から書いたものです。

説明

documentation

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] and safe-outputs: submit-pull-request-review (not add-comment).

Suggested fix

  1. Replace all links/labels pointing to 15-conditional-logic.md as "PR Code Reviewer" with links to 14b-pr-reviewer-workflow.md, using its real title "Build a PR Reviewer with an Agent and Skill" (or "PR Auto-Reviewer" per the README).
  2. Change "Step 11c" to "Step 14b".
  3. 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
主要言語
JavaScript
スター
52
フォーク
26
平均マージ
12時間 58分
マージ済み PR（30日）
15

環境構築

Codespaces で開く

このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。

  • Dockerfile・Docker Compose ファイルなし
  • プルリクエストのテンプレートなし
  • コントリビューションガイドなし

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

githubnext/gh-aw-workshop のほかの issue

githubnext/gh-aw-workshop の issue をすべて見る

似ている issue

JavaScript の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。