-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Task library documentation and structure clean up #3360
Conversation
Co-authored-by: Jim Crist-Harif <jcrist@users.noreply.github.com>
Co-authored-by: Jim Crist-Harif <jcrist@users.noreply.github.com>
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.
Overall this looks good to me.
.flake8
Outdated
@@ -11,7 +11,7 @@ ignore = | |||
E203, | |||
# Comparing types instead of isinstance | |||
E721, | |||
# Assign a lambda | |||
# Assign a lam\bda |
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.
why?
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.
why not?
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.
Left some comments - we can merge this as-is and revisit the comments for the full post + as follow-up PRs. In general I think we might want to expand on "design" for tasks (e.g., what to include in init vs run) and maybe provide some examples but this is still a great start
Was just in a conversation with @kingsleyb and I realized we need to document things like our desired pattern for interacting with secrets and also some more information WRT init and run kwargs, stand by |
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.
Two small suggestions
Co-authored-by: Chris White <chris@prefect.io>
Summary
This PR cleans up the task library documentation and moves around some files that were previously in
/contrib
.Changes
/contrib
prefect.contrib.tasks
toprefect.tasks
(warning raise if imported from old location)Importance
Task library was missing a true home for documentation. The
contrib
directory is not useful and it doesn't benefit having some tasks separate from the rest.Checklist
This PR:
changes/
directory (if appropriate)docs/outline.toml
for API reference docs (if appropriate)