docs: Add doc-impact review process to team workflow#292
Merged
bradygaster merged 1 commit intobradygaster:mainfrom Mar 9, 2026
Merged
docs: Add doc-impact review process to team workflow#292bradygaster merged 1 commit intobradygaster:mainfrom
bradygaster merged 1 commit intobradygaster:mainfrom
Conversation
Contributor
Author
|
@bradygaster — doc-impact review process PR. Adds PAO as automatic doc reviewer for all PRs. Part of the new-customer docs improvement series (PRs #286, #288, #290, #292). |
Add doc-impact review as a team convention: - decisions.md: Every PR evaluated for documentation impact - PAO charter: Doc-impact review added as hard rule - routing.md: Auto-involve PAO for doc review on PR work Closes bradygaster#291 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
diberry
force-pushed
the
squad/new-customer-docs-4
branch
from
3f503df to
c74ec9b
Compare
bradygaster
added a commit
that referenced
this pull request
Mar 9, 2026
…post - Port @diberry's installation decision tree (PR #288) and .squad/ explainer (PR #290) into Astro docs - Apply Brady's review fixes: sidebar scroll-to-active, CLI text updates, logo fix, Copilot CLI callouts, CI/CD cron safety warning - Update CONTRIBUTORS.md with @IEvangelist (Astro docs rewrite) and @diberry (4 PRs) - Add blog post 028: New Docs Site — Built by the Community Merges community work from: - PR #293 by @IEvangelist (Astro docs rewrite) - PR #286, #288, #290, #292 by @diberry (docs improvements) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
bradygaster
added a commit
that referenced
this pull request
Mar 9, 2026
- Add David Pine (@IEvangelist) to Code Contributors — PRs #293, #298 - Add Dina Berry (@diberry) to Code Contributors — PRs #286, #288, #290, #292 - Update Tamir Dresher with recent PRs #278-280, #283 - Add Dale Kirby to Code Contributors — PR #243 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
bradygaster
added a commit
that referenced
this pull request
Mar 9, 2026
* docs(ai-team): session log and orchestration records Session: 2026-03-09-docs-push-contributors Requested by: Brady Changes: - Logged PAO orchestration: blog 028 update, CONTRIBUTORS.md refresh - Logged FIDO orchestration: test baseline verification (22 passing) - Logged session summary: docs push workflow with PR #300 merge - No decisions inbox entries to merge * docs: credit Pine, Dina, Tamir, Kirby on docs contributor page - Add David Pine (@IEvangelist) to Code Contributors — PRs #293, #298 - Add Dina Berry (@diberry) to Code Contributors — PRs #286, #288, #290, #292 - Update Tamir Dresher with recent PRs #278-280, #283 - Add Dale Kirby to Code Contributors — PR #243 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
diberry
pushed a commit
to diberry/squad
that referenced
this pull request
Mar 10, 2026
…gaster#307) * docs(ai-team): session log and orchestration records Session: 2026-03-09-docs-push-contributors Requested by: Brady Changes: - Logged PAO orchestration: blog 028 update, CONTRIBUTORS.md refresh - Logged FIDO orchestration: test baseline verification (22 passing) - Logged session summary: docs push workflow with PR bradygaster#300 merge - No decisions inbox entries to merge * docs: credit Pine, Dina, Tamir, Kirby on docs contributor page - Add David Pine (@IEvangelist) to Code Contributors — PRs bradygaster#293, bradygaster#298 - Add Dina Berry (@diberry) to Code Contributors — PRs bradygaster#286, bradygaster#288, bradygaster#290, bradygaster#292 - Update Tamir Dresher with recent PRs bradygaster#278-280, bradygaster#283 - Add Dale Kirby to Code Contributors — PR bradygaster#243 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds a doc-impact review step to the team workflow so every PR is evaluated for documentation changes.
Changes
squadWhen a New Version is Released #7 — auto-involve PAO for doc reviewLabels
documentation,P0Closes #291
cc @bradygaster @dfberry