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

[dagster-airlift] [api-docs] additional stuff for loader #25690

Merged
merged 1 commit into from
Nov 7, 2024

Conversation

dpeng817
Copy link
Contributor

@dpeng817 dpeng817 commented Nov 1, 2024

Summary & Motivation

Additional docstrings + top-level stuff for loader fxn

How I Tested These Changes

Eyes

Copy link

github-actions bot commented Nov 1, 2024

Deploy preview for dagster-docs ready!

Preview available at https://dagster-docs-l4n069uau-elementl.vercel.app
https://dpeng817-docs-loader-additional.dagster.dagster-docs.io

Direct link to changed pages:

@dpeng817 dpeng817 force-pushed the dpeng817/docs/loader_additional branch from d0e4ced to 1e21c40 Compare November 1, 2024 22:01
@dpeng817 dpeng817 force-pushed the dpeng817/docs/loader_additional branch from 1e21c40 to 8473c54 Compare November 1, 2024 22:09
@dpeng817 dpeng817 force-pushed the dpeng817/docs/loader_additional branch from 8473c54 to b100602 Compare November 1, 2024 23:08
@dpeng817 dpeng817 force-pushed the dpeng817/docs/loader_additional branch from b100602 to 9afa6e0 Compare November 1, 2024 23:16
@dpeng817 dpeng817 force-pushed the dpeng817/docs/loader_additional branch from 9afa6e0 to 121db4c Compare November 1, 2024 23:20
@dpeng817 dpeng817 force-pushed the dpeng817/docs/loader_additional branch from 121db4c to 74ed6d5 Compare November 6, 2024 18:59
@dpeng817 dpeng817 force-pushed the dpeng817/docs/loader_additional branch from 74ed6d5 to 4b609db Compare November 6, 2024 21:55
@dpeng817 dpeng817 force-pushed the dpeng817/docs/loader_additional branch from 4b609db to d636cca Compare November 7, 2024 19:47
@dpeng817 dpeng817 force-pushed the dpeng817/docs/loader_additional branch from d636cca to cf7d094 Compare November 7, 2024 19:52
@dpeng817 dpeng817 force-pushed the dpeng817/docs/loader_additional branch from cf7d094 to f9b64bc Compare November 7, 2024 19:54
@dpeng817 dpeng817 force-pushed the dpeng817/docs/loader_additional branch from f9b64bc to 15fa119 Compare November 7, 2024 21:17
@dpeng817 dpeng817 force-pushed the dpeng817/docs/loader_additional branch from 15fa119 to b838ce3 Compare November 7, 2024 21:21
Base automatically changed from dpeng817/docs/loader to master November 7, 2024 21:27
@dpeng817 dpeng817 force-pushed the dpeng817/docs/loader_additional branch from b838ce3 to b627bfd Compare November 7, 2024 21:29
@dpeng817 dpeng817 merged commit 6d3bcf7 into master Nov 7, 2024
0 of 3 checks passed
@dpeng817 dpeng817 deleted the dpeng817/docs/loader_additional branch November 7, 2024 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants