docs: fix steps: placement in Deterministic Multi-Repo Workflows example#18143
docs: fix steps: placement in Deterministic Multi-Repo Workflows example#18143
steps: placement in Deterministic Multi-Repo Workflows example#18143Conversation
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
steps: placement in Deterministic Multi-Repo Workflows example
There was a problem hiding this comment.
Pull request overview
This PR fixes a critical YAML structure error in the MultiRepoOps documentation where steps: was incorrectly nested under engine:, causing compilation to fail with "Unknown property: steps". The fix correctly moves steps: to be a top-level frontmatter field as specified in the schema.
Changes:
- Corrected YAML structure in the "Deterministic Multi-Repo Workflows" example by moving
steps:from underengine:to top-level frontmatter - Regenerated smoke-copilot.lock.yml workflow file (includes unrelated infrastructure update)
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| docs/src/content/docs/patterns/multi-repo-ops.md | Fixed incorrect YAML structure by moving steps: array from nested under engine: to top-level frontmatter position |
| .github/workflows/smoke-copilot.lock.yml | Auto-generated lock file with updated compilation output (includes new MCP configuration cleanup step) |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| - name: Clear MCP configuration for detection | ||
| if: always() && steps.detection_guard.outputs.run_detection == 'true' | ||
| run: | | ||
| rm -f /tmp/gh-aw/mcp-config/mcp-servers.json | ||
| rm -f /home/runner/.copilot/mcp-config.json | ||
| rm -f "$GITHUB_WORKSPACE/.gemini/settings.json" |
There was a problem hiding this comment.
This auto-generated lock file includes a new step "Clear MCP configuration for detection" that wasn't present before. Since this file is automatically generated from smoke-copilot.md (which doesn't contain this step definition), this change appears to come from updates to the compilation infrastructure itself.
While this may be expected if the compiler was updated, the PR description doesn't mention this change. If this is an intentional infrastructure update that affects all compiled workflows, it would be helpful to note this in the PR description. If this change is unintentional or if only the documentation fix should be included, consider regenerating the lock file without this additional change.
The "Deterministic Multi-Repo Workflows" code example in the MultiRepoOps docs had
steps:incorrectly nested underengine:, causing compilation to fail withUnknown property: steps.Change
docs/src/content/docs/patterns/multi-repo-ops.md: Movedsteps:from underengine:to a top-level frontmatter key.Original prompt
🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.