<#315 remove trigger_time as default argument in t...
# flyte-github
a
#315 remove trigger_time as default argument in triggers Pull request opened by dansola • Changed parameter from
trigger_time_input_key: str = "trigger_time"
to
trigger_time_input_key: str | None = None
• Only add the trigger time input if
trigger_time_input_key is not None
• Tested by creating a validation script that verified all five trigger convenience methods (daily(), hourly(), minutely(), weekly(), monthly()) correctly return empty inputs by default and properly add
trigger_time
inputs only when explicitly specified. • ran the linter that changes a couple exiting files flyteorg/flyte-sdk