-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[docs] - Update Schedule API docs (DOC-236) #21838
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Generally looks good to me, but looks like there was a rough merge that accidentally pulled in some old (or new?) changes in some of the actual codebase! Feel free to hit re-review when that merge is fixed.
Also left a couple comments to hone in on some edges.
python_modules/dagster/dagster/_core/definitions/decorators/schedule_decorator.py
Outdated
Show resolved
Hide resolved
python_modules/dagster/dagster/_core/definitions/time_window_partitions.py
Outdated
Show resolved
Hide resolved
This reverts commit 1e611db.
…r-io/dagster into erin/doc-236-api-schedule
python_modules/dagster/dagster/_core/definitions/time_window_partitions.py
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! thanks for dealing with git hell.
Summary & Motivation
This PR updates the Schedule API docs by adding information to the reference page itself as well as some of the API descriptions.
How I Tested These Changes
eyes + local, ruff + BK