Summary This PR fixes the GitHub emoji shortcode guideline violation in 9 side-quest workshop files. The guideline states: Write checkpoint headings as ## :whitecheckmark: Checkpoint so they use the same browser-independent rendering as other icons. All 9 files used the raw Unicode ✅ character instead of the :whitecheckmark: shortcode. Files changed | File | Fix applied | |------|-------------| | workshop/side-quest-13-03-pr-checklist-pattern.md | ## ✅ Checkpoint → ## :whitecheckmark: Checkpoint | | workshop/side-quest-13-04-token-optimization.md | ## ✅ Checkpoint → ## :whitecheckmark: Checkpoint | | workshop/side-quest-15-01-expressions-and-contexts.md | ## ✅ Checkpoint → ## :whitecheckm...
Guidelines Enforcer Reviewed 10 changed markdown files in PR #2716 (workshop/17-add-mcp-tools.md through workshop/26-manage-costs-and-budgets.md) — no guideline violations found. Checks performed: No numbered headers (headings with leading digits) found in any file ✓ Alert callouts: all single-line callouts use plain blockquotes; all multi-line callouts use <details> with <summary> ✓ Alert severity: only [!TIP], [!NOTE], and [!IMPORTANT] used — no inappropriate [!WARNING] or [!CAUTION] ✓ No raw cron syntax in any .md workflow file ✓ No Node.js prerequisites or node --version checks ✓ All workshop step files end with ## :whitecheckmark: Checkpoint containing a markdown checklist ✓ Voice an...
File reviewed workshop/side-quest-13-04-token-optimization.md Violations No standalone "See..." footers — links belong inline Offending text (line 35): Offending text (line 116): Suggested fix: Remove these standalone blockquote lines. Embed the links inline in the preceding prose — for example, change the gh aw audit command reference to use an inline link to the audit reference page, and fold the Cost Management link into the preceding bullet list sentence. [!WARNING] <details> <summary>Firewall blocked 1 domain</summary> The following domain was blocked by the firewall during workflow execution: - awmgmcpg To allow these domains, add them to the network.allowed list in your workflow fr...
File reviewed workshop/14b-pr-reviewer-workflow.md Violations No "See Also" sections — documentation links belong inline Offending text: Suggested fix: Remove the ## Go Further: PR Reviewer Patterns section entirely. This is a de-facto "See Also" hub placed after the checkpoint and Next link. Instead, integrate references to individual side quests as natural inline mentions within the body of the step (for example, when describing the labeler capability, add a parenthetical link to the labeler pattern side quest). [!WARNING] <details> <summary>Firewall blocked 1 domain</summary> The following domain was blocked by the firewall during workflow execution: - awmgmcpg To allow these domains, ...