Skip to content

docs: fix schedule frequency list in update-workflow validation checklist#1629

Merged
jamesadevine merged 1 commit into
mainfrom
docs/fix-schedule-frequency-checklist-07c0e89ae012a0d7
Jul 22, 2026
Merged

docs: fix schedule frequency list in update-workflow validation checklist#1629
jamesadevine merged 1 commit into
mainfrom
docs/fix-schedule-frequency-checklist-07c0e89ae012a0d7

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Agent Documentation Update

Findings

Area Issue File(s)
accuracy Validation checklist item #3 omits weekly (any day, scattered time) from the list of valid schedule frequencies; also omits every N hours (long form of every Nh) prompts/update-ado-agentic-workflow.md

Applied Fixes

  • Added weekly (plain, no on <day>) to the valid frequency list — this form is accepted by parse_weekly_schedule() in src/fuzzy_schedule.rs and documented in docs/schedule-syntax.md
  • Added every N hours alongside every Nh to show both short and long forms are accepted

Notes

Verified against src/fuzzy_schedule.rs: parse_weekly_schedule returns Ok(FuzzySchedule::Weekly { day: None, ... }) when called with no arguments, so plain weekly is valid. Cross-checked with docs/schedule-syntax.md which shows schedule: weekly # Any day, scattered time as a canonical example.

The create-ado-agentic-workflow.md prompt's frequency table already includes weekly — only the update prompt's checklist was missing it.


Created by the agent-documentation maintainer workflow.

Generated by Agent Documentation Maintainer · 125 AIC · ⌖ 11.8 AIC · ⊞ 8.7K ·

…list

Plain `weekly` (any day, scattered time) is a valid fuzzy schedule
frequency — supported in src/fuzzy_schedule.rs and documented in
docs/schedule-syntax.md — but was omitted from the validation checklist
in prompts/update-ado-agentic-workflow.md.

Also adds the long form `every N hours` alongside the short `every Nh`
for completeness.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@jamesadevine
jamesadevine marked this pull request as ready for review July 22, 2026 20:54
@jamesadevine
jamesadevine merged commit 89b1a9f into main Jul 22, 2026
3 of 5 checks passed
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