Skip to content

docs(site): remove invalid workflow_dispatch trigger from quick-start example#706

Merged
jamesadevine merged 1 commit into
mainfrom
docs/fix-workflow-dispatch-in-quick-start-929dddba3256ceea
May 22, 2026
Merged

docs(site): remove invalid workflow_dispatch trigger from quick-start example#706
jamesadevine merged 1 commit into
mainfrom
docs/fix-workflow-dispatch-in-quick-start-929dddba3256ceea

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

Summary

  • Removes on: workflow_dispatch: from the manual quick-start example — workflow_dispatch is a GitHub Actions trigger key that has no meaning in ado-aw and is silently ignored at compile time.
  • Adds a callout note explaining the ADO manual-queuing model: all ADO pipelines are always manually queueable; on: is only needed for automatic triggers (schedules, PR events, pipeline completion).

Changes

  • site/src/content/docs/setup/quick-start.mdx:
    • Removed on: workflow_dispatch: from the minimal agent file example.
    • Added a :::note[Manual queuing] callout explaining what on: is for and that it is not needed for manual runs.

Accuracy checks

  • src/compile/types.rs OnConfig struct: fields are schedule, pipeline, pr — no workflow_dispatch field. OnConfig lacks deny_unknown_fields so unknown keys are silently dropped during deserialization.
  • Confirmed: same class of bug was fixed in creating-agents.mdx in PR docs(site): fix trigger field names in creating-agents guide #703.

Validation

  • cd site && npm ci && npm run build — 29 pages built, all internal links valid.

Created by the docs-writer workflow.

Generated by Docs Writer · ● 17.2M ·

… example

workflow_dispatch is a GitHub Actions key with no meaning in ado-aw.
OnConfig only accepts schedule, pipeline, and pr — unknown keys are
silently dropped. The minimal example now omits on: entirely and a
callout explains the ADO manual-queuing model.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@jamesadevine jamesadevine marked this pull request as ready for review May 22, 2026 21:25
@jamesadevine jamesadevine merged commit 2be2d2c into main May 22, 2026
@jamesadevine jamesadevine deleted the docs/fix-workflow-dispatch-in-quick-start-929dddba3256ceea branch May 22, 2026 21:26
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.

1 participant