Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use schedule param rather than timetable in Timetables docs #29255

Merged
merged 1 commit into from
Jan 31, 2023

Conversation

josh-fell
Copy link
Contributor

As of 2.4.0, the timetable DAG param has been deprecated with preference for schedule. The docs for Timetables still reference timetable in the code examples.

P.S. Small "bonus" added of removing the as dag: registration of DAGs initialized with a content manager too.

As of 2.4.0, the `timetable` DAG param has been deprecated with preference for `schedule`. The docs for Timetables still reference `timetable` in the code examples.

P.S. Small "bonus" added of removing the `as dag:` registration of DAGs initialized with a content manager too.
@BasPH BasPH merged commit 51d9633 into apache:main Jan 31, 2023
@josh-fell josh-fell deleted the timetable-docs branch January 31, 2023 13:35
@pierrejeambrun pierrejeambrun added the type:doc-only Changelog: Doc Only label Feb 27, 2023
@pierrejeambrun pierrejeambrun added this to the Airflow 2.5.2 milestone Feb 27, 2023
pierrejeambrun pushed a commit that referenced this pull request Mar 7, 2023
As of 2.4.0, the `timetable` DAG param has been deprecated with preference for `schedule`. The docs for Timetables still reference `timetable` in the code examples.

P.S. Small "bonus" added of removing the `as dag:` registration of DAGs initialized with a content manager too.

(cherry picked from commit 51d9633)
pierrejeambrun pushed a commit that referenced this pull request Mar 8, 2023
As of 2.4.0, the `timetable` DAG param has been deprecated with preference for `schedule`. The docs for Timetables still reference `timetable` in the code examples.

P.S. Small "bonus" added of removing the `as dag:` registration of DAGs initialized with a content manager too.

(cherry picked from commit 51d9633)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants