Add temporal schedule list-matching-times command …#1047
Conversation
…tScheduleMatchingTimes RPC in the CLI. Allows users to preview when a schedule will fire within a given time range without executing workflows. Closes temporalio#1030
|
@NasitSony thank you for your contribution! Could to add a functional test for this command? 1 with the human readable output and 1 with json? Also, there's a new PR template, could you follow it? |
|
@chaptersix Thanks for the feedback! I'll add the functional tests and update to the new PR template. |
|
@chaptersix Added functional tests for text and JSON output, and fixed JSON output to use PrintStructured for proper array format. Also updated the PR description to follow the new template. Ready for review. |
Update command description to reference schedule spec and actions instead of Workflow Executions, and use "(Experimental feature)" to match CLI conventions. Fix JSON output to pass through raw RPC response and text output to use table format with RFC3339 timestamps. Simplify tests with deterministic calendar spec.
Related issuesCloses #1030 What changed?Adds ChecklistStability
Design
Help text (see style guide at the top of
Behavior
Tests
Manual testsSetup Happy path Error case Composition |
|
@NasitSony, I went ahead and made some corrections while I have time to review. Thanks again! |
Implements the ListScheduleMatchingTimes RPC in the CLI.
Allows users to preview when a schedule will fire within
a given time range without executing workflows.
What was changed
Added
temporal schedule list-matching-timescommand implementing theListScheduleMatchingTimesRPC. Added functional tests for both text and JSON output. Fixed JSON output to usePrintStructuredfor proper array format.Why?
The Temporal CLI was missing a command for the
ListScheduleMatchingTimesRPC. Users can now preview when a schedule would fire within a given time range without executing workflows.Checklist