Skip to content

docs: clarify first-steps review access and app workflows - #196

Merged
jamesmontemagno merged 1 commit into
mainfrom
motz-copilot-review-eligibility
Oct 5, 2026
Merged

jamesmontemagno merged 1 commit into
mainfrom
motz-copilot-review-eligibility

Conversation

@jamesmontemagno

Copy link
Copy Markdown
Collaborator

Summary

Prevent learners from reaching unavailable PR review features or following app instructions that do not match a local-folder session and its approval settings. Clarify Copilot Free review eligibility, provide a manual-review alternative, and correct session inspection, /init input, and publishing guidance across English and all five translations.

  • Add review-access caveats to app and VS Code prerequisites and Lesson 8, distinguishing Copilot Free from Copilot Student and organization-paid access.
  • Limit pre-Git session inspection to project, path, and context usage; identify the illustration as a later Git-backed example and link to publishing.
  • Use /init create simple rules for the project in app, CLI, and VS Code execution steps, explaining the missing-input error in the app and CLI lessons.
  • Explain that publishing may prompt or proceed automatically depending on approval settings, and move the safety review before the publishing prompt.

Type of change

  • Workshop content (lesson Markdown, images)
  • Site shell (website/ Astro + Starlight wrapper)
  • Copilot configuration (.github/copilot-instructions.md, instructions, agents, skills)
  • Repo housekeeping (CI, dependabot, README, license)
  • Other: N/A

Verification

  • Type checks and a clean site build pass.
  • Affected pages and translations match their source and intended routes.
  • Offline internal link checks pass.
  • Changed external links have been checked separately.

Commands and results:

  • npm.cmd ci --no-audit --no-fund in website: passed; restored the existing lockfile dependencies after the initial verification attempt reported missing Astro.
  • & .\.github\skills\build-and-verify-docs\scripts\verify.ps1: passed, including npm run check:all, a clean npm run build, and offline Lychee validation with 0 link errors. Type checks reported 0 errors, 0 warnings, and 4 existing hints.
  • git diff --check and git diff --cached --check: passed.

Source-to-built HTML inspection passed for all 54 changed lessons: expected routes, titles, locale languages, command examples, callout text, and updated reference destinations. Reviewed parallel lessons and all five translations for content alignment. Checked the new GitHub code-review documentation destination and its eligibility guidance separately.

Screenshots

N/A. No image or layout changes; the existing session-details illustration is now explicitly labeled as a later Git-backed example.

Notes for reviewers

The organization-paid review exception applies on GitHub.com and does not grant IDE review access. Bare /init remains as the command name in headings and conceptual descriptions, while execution steps include a prompt.

Browser QA was not run because Playwright MCP tools were unavailable. The existing plain-div admonition rendering is unchanged; this PR does not change site-shell styling.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings October 5, 2026 21:37

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟢 Approval recommended

The workflows, links, commands, and localized counterparts are complete and consistently aligned.

Review effort: Balanced
Findings: None

What changed in this PR

Clarifies feature access and corrects first-steps workflows across English and five localized editions.

Changes:

  • Documents code-review eligibility and manual alternatives.
  • Corrects session inspection, /init, and publishing guidance.
  • Keeps all localized lessons aligned.
File Description
docs/​first-steps/​copilot-app/​0-prerequisites.md Adds review-access prerequisite.
docs/​first-steps/​copilot-app/​3-inspect-and-test.md Corrects local-session inspection.
docs/​first-steps/​copilot-app/​4-project-instructions.md Adds required /init input.
docs/​first-steps/​copilot-app/​5-publish.md Clarifies approval behavior.
docs/​first-steps/​copilot-app/​8-review-loop.md Adds manual-review alternative.
docs/​first-steps/​copilot-cli/​2-project-instructions.md Adds required /init input.
docs/​first-steps/​vscode/​0-prerequisites.md Adds review-access prerequisite.
docs/​first-steps/​vscode/​2-project-instructions.md Updates the /init command.
docs/​first-steps/​vscode/​8-review-and-merge.md Clarifies review eligibility.
docs/​es-es/​first-steps/​copilot-app/​0-prerequisites.md Localizes review prerequisite.
docs/​es-es/​first-steps/​copilot-app/​3-inspect-and-test.md Localizes session corrections.
docs/​es-es/​first-steps/​copilot-app/​4-project-instructions.md Localizes /init guidance.
docs/​es-es/​first-steps/​copilot-app/​5-publish.md Localizes approval guidance.
docs/​es-es/​first-steps/​copilot-app/​8-review-loop.md Localizes review alternative.
docs/​es-es/​first-steps/​copilot-cli/​2-project-instructions.md Localizes /init guidance.
docs/​es-es/​first-steps/​vscode/​0-prerequisites.md Localizes review prerequisite.
docs/​es-es/​first-steps/​vscode/​2-project-instructions.md Localizes the /init command.
docs/​es-es/​first-steps/​vscode/​8-review-and-merge.md Localizes eligibility guidance.
docs/​ja-jp/​first-steps/​copilot-app/​0-prerequisites.md Localizes review prerequisite.
docs/​ja-jp/​first-steps/​copilot-app/​3-inspect-and-test.md Localizes session corrections.
docs/​ja-jp/​first-steps/​copilot-app/​4-project-instructions.md Localizes /init guidance.
docs/​ja-jp/​first-steps/​copilot-app/​5-publish.md Localizes approval guidance.
docs/​ja-jp/​first-steps/​copilot-app/​8-review-loop.md Localizes review alternative.
docs/​ja-jp/​first-steps/​copilot-cli/​2-project-instructions.md Localizes /init guidance.
docs/​ja-jp/​first-steps/​vscode/​0-prerequisites.md Localizes review prerequisite.
docs/​ja-jp/​first-steps/​vscode/​2-project-instructions.md Localizes the /init command.
docs/​ja-jp/​first-steps/​vscode/​8-review-and-merge.md Localizes eligibility guidance.
docs/​ko-kr/​first-steps/​copilot-app/​0-prerequisites.md Localizes review prerequisite.
docs/​ko-kr/​first-steps/​copilot-app/​3-inspect-and-test.md Localizes session corrections.
docs/​ko-kr/​first-steps/​copilot-app/​4-project-instructions.md Localizes /init guidance.
docs/​ko-kr/​first-steps/​copilot-app/​5-publish.md Localizes approval guidance.
docs/​ko-kr/​first-steps/​copilot-app/​8-review-loop.md Localizes review alternative.
docs/​ko-kr/​first-steps/​copilot-cli/​2-project-instructions.md Localizes /init guidance.
docs/​ko-kr/​first-steps/​vscode/​0-prerequisites.md Localizes review prerequisite.
docs/​ko-kr/​first-steps/​vscode/​2-project-instructions.md Localizes the /init command.
docs/​ko-kr/​first-steps/​vscode/​8-review-and-merge.md Localizes eligibility guidance.
docs/​pt-br/​first-steps/​copilot-app/​0-prerequisites.md Localizes review prerequisite.
docs/​pt-br/​first-steps/​copilot-app/​3-inspect-and-test.md Localizes session corrections.
docs/​pt-br/​first-steps/​copilot-app/​4-project-instructions.md Localizes /init guidance.
docs/​pt-br/​first-steps/​copilot-app/​5-publish.md Localizes approval guidance.
docs/​pt-br/​first-steps/​copilot-app/​8-review-loop.md Localizes review alternative.
docs/​pt-br/​first-steps/​copilot-cli/​2-project-instructions.md Localizes /init guidance.
docs/​pt-br/​first-steps/​vscode/​0-prerequisites.md Localizes review prerequisite.
docs/​pt-br/​first-steps/​vscode/​2-project-instructions.md Localizes the /init command.
docs/​pt-br/​first-steps/​vscode/​8-review-and-merge.md Localizes eligibility guidance.
docs/​zh-cn/​first-steps/​copilot-app/​0-prerequisites.md Localizes review prerequisite.
docs/​zh-cn/​first-steps/​copilot-app/​3-inspect-and-test.md Localizes session corrections.
docs/​zh-cn/​first-steps/​copilot-app/​4-project-instructions.md Localizes /init guidance.
docs/​zh-cn/​first-steps/​copilot-app/​5-publish.md Localizes approval guidance.
docs/​zh-cn/​first-steps/​copilot-app/​8-review-loop.md Localizes review alternative.
docs/​zh-cn/​first-steps/​copilot-cli/​2-project-instructions.md Localizes /init guidance.
docs/​zh-cn/​first-steps/​vscode/​0-prerequisites.md Localizes review prerequisite.
docs/​zh-cn/​first-steps/​vscode/​2-project-instructions.md Localizes the /init command.
docs/​zh-cn/​first-steps/​vscode/​8-review-and-merge.md Localizes eligibility guidance.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@jamesmontemagno
jamesmontemagno merged commit aeebea5 into main Oct 5, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants