You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/08/01 03:11:07 UTC

[GitHub] [airflow] mai-nakagawa commented on a diff in pull request #23662: Implement CronTriggerTimetable

mai-nakagawa commented on code in PR #23662:
URL: https://github.com/apache/airflow/pull/23662#discussion_r934105910


##########
docs/apache-airflow/concepts/timetable.rst:
##########
@@ -55,15 +55,17 @@ Built In Timetables
 Airflow comes with several common timetables built in to cover the most common use cases. Additional timetables
 may be available in plugins.
 
+.. _CronTriggerTimetable
+
 CronTriggerTimetable
 ^^^^^^^^^^^^^^^^^^^^^^^^^
 
-Another timetable which accepts a cron expression. It behaves more compliant with standard cron than ``CronDataIntervalTimetable``.
+A cron compliant timetable.

Review Comment:
   Stopped using "cron compliant" and created a separated section to tell the differences between the two timetables. Hope it makes more sense for the readers: https://github.com/apache/airflow/pull/23662/commits/92ab13b3a73aa746ce9afd25fc63c29feb6acef8



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org