Conversation
|
✅ Smoke CI completed successfully! |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Pull request overview
Adds new documentation covering organization-scale workflow practices and introduces the CorrectionOps design pattern for improving workflows using trusted human corrections (without retraining models).
Changes:
- Add new CorrectionOps pattern documentation page.
- Add new Organization Practices section with pages for Safe Rollout and Sharing Workflows.
- Update glossary, sidebar navigation, and redirects to surface the new docs and route legacy URLs.
Show a summary per file
| File | Description |
|---|---|
| docs/src/content/docs/reference/glossary.md | Adds glossary entries for CorrectionOps and organization-practice terms. |
| docs/src/content/docs/patterns/correction-ops.md | New pattern page describing CorrectionOps and a starter architecture. |
| docs/src/content/docs/organization-practices/index.mdx | New landing page for the Organization Practices docs section. |
| docs/src/content/docs/organization-practices/safe-rollout.md | New guidance on staged → shadow → direct-write rollout progression. |
| docs/src/content/docs/organization-practices/sharing-workflows.md | New guidance on workflow sharing models and governance questions. |
| docs/astro.config.mjs | Adds redirects for legacy shadow URLs and adds sidebar entries for new sections/pages. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comments suppressed due to low confidence (2)
docs/src/content/docs/patterns/correction-ops.md:24
- Spelling is inconsistent with the rest of the docs: this repo predominantly uses US spelling “labeling” (single “l”). Consider changing “labelling” → “labeling” here.
Typical fits include labelling and classification, routing and prioritization, moderation and approvals, and summaries or recommendations that humans later correct.
docs/src/content/docs/patterns/correction-ops.md:279
- Spelling is inconsistent with the rest of the docs: this repo predominantly uses US spelling “labeling” (single “l”). Consider changing “labelling” → “labeling” here.
Discussion labelling, routing, moderation, prioritization, approvals, and summaries can all reuse this shape. The production object changes, but the CorrectionOps setup does not.
- Files reviewed: 6/6 changed files
- Comments generated: 4
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
✅ Smoke CI completed successfully! |
This comment has been minimized.
This comment has been minimized.
|
Hey
If you'd like to split this up with a coding agent, here's a ready-to-use prompt:
|
|
@copilot Create a new PR for the CorrectionOps design pattern and keep this PR for organization practices (guidance) only. |
Agent-Logs-Url: https://github.com/github/gh-aw/sessions/e3c9ad61-cf1e-460e-bf6b-11131493b22d Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
|
@copilot Remove Organization Practices from this PR. |
Agent-Logs-Url: https://github.com/github/gh-aw/sessions/c759f2a5-b6aa-4540-9253-7c862e444e68 Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
Done in 11e81c2. Removed the Organization Practices files (index.mdx, safe-rollout.md, sharing-workflows.md), their sidebar entries and redirects in astro.config.mjs, and the Organization Practices, Safe Rollout, and Shadow Evaluation glossary entries. The CorrectionOps content remains in this PR. |
|
@copilot Update text: For sharing GitHub Agentic Workflows, they should support a few layers:
AW supports imports: for shared tools, MCP servers, steps, jobs, and prompt snippets. AW supports uses / with for parameterized imports.
|
…r model Agent-Logs-Url: https://github.com/github/gh-aw/sessions/3331b9b4-51cd-43d8-965e-8863bf5f3ab9 Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
Done in f269507. The sharing-workflows.md guide has been rewritten around the seven layers you described (whole workflow install, reusable imports, parameterized templates, versioning/update flow, private/internal controls, import caching and lock behavior, and cross-repo execution), with the recommended enterprise pattern (central |
Uh oh!
There was an error while loading. Please reload this page.