Skip to content

Commit a4f5edb

Browse files
CopilotEMaher
andauthored
feat: revise air-gapped Azure DevOps doc per PR feedback
- Make local npm registry (Azure Artifacts) the primary approach - Demote tarball flow to fallback section - Remove redundant npm and transfer mechanism prerequisites - Add official Microsoft docs links (agents, feeds, on-prem, sovereign clouds) - Replace brittle embedded YAML with edit guidance referencing generated files - Add sovereign clouds and on-premises section Closes #44 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: EMaher <9244742+EMaher@users.noreply.github.com>
1 parent af26aaf commit a4f5edb

2 files changed

Lines changed: 115 additions & 191 deletions

File tree

.squad/agents/docwriter/history.md

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -131,3 +131,21 @@ Documentation authoring (2026-04-30 to 2026-05-17): 3-phase plan with 28 user-fa
131131
**Patterns:** Examples-first, Mermaid workflows, relative links, search-optimized errors, progressive disclosure
132132

133133
**Gotchas:** Auth flags set env vars (credential precedence). Overrides: names consistent, properties differ. `--commit-id`/`--delete-unmatched` exclusive.
134+
135+
### 2026-06-11: Air-Gapped Azure DevOps — PR Feedback Revision
136+
137+
**Context:** Revised `docs/walkthrough/air-gapped-azure-devops.md` based on PR review comments.
138+
139+
**Changes:**
140+
- Made local npm registry (Azure Artifacts) the primary/default approach; tarball flow demoted to fallback section.
141+
- Removed redundant "npm 10+" prerequisite (implied by Node.js 22.x).
142+
- Removed "transfer mechanism" prerequisite row (not needed when local registry is primary).
143+
- Added official Microsoft docs links: self-hosted agents, Azure Artifacts npm feeds, Azure DevOps Server on-prem, sovereign cloud identity endpoints.
144+
- Replaced full embedded pipeline YAML with concise edit table referencing generated files (`pipelines/run-extractor.yaml`, `pipelines/run-publisher.yaml`).
145+
- Added "Sovereign Clouds and On-Premises" section with doc links.
146+
- Simplified architecture diagram to reflect local registry flow.
147+
148+
**Learnings:**
149+
- PR reviewers strongly prefer docs that match the air-gapped reality: local registries over manual transfers. Azure Artifacts is a natural fit since it's bundled with Azure DevOps Server.
150+
- Large embedded YAML examples are fragile — they drift from generated templates. Better to describe minimal edits to the generated files.
151+
- Always link to official Microsoft docs for infrastructure setup (agents, feeds, sovereign clouds) rather than inlining instructions that will go stale.

0 commit comments

Comments
 (0)