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

[5/n][dagster-dlt] Update dlt docs to use DagsterDltTranslator.get_asset_spec #27244

Merged
merged 1 commit into from
Jan 21, 2025

Conversation

maximearmstrong
Copy link
Contributor

@maximearmstrong maximearmstrong commented Jan 21, 2025

Summary & Motivation

As title.

@maximearmstrong maximearmstrong force-pushed the maxime/dlt-asset-spec-4 branch from f3ddca8 to 1540c42 Compare January 21, 2025 19:22
@maximearmstrong maximearmstrong force-pushed the maxime/dlt-asset-spec-5 branch from 39d0d26 to 322539c Compare January 21, 2025 19:22
@maximearmstrong maximearmstrong marked this pull request as ready for review January 21, 2025 19:32
@maximearmstrong maximearmstrong self-assigned this Jan 21, 2025
@maximearmstrong maximearmstrong requested review from cmpadden and OwenKephart and removed request for neverett January 21, 2025 19:32
@neverett
Copy link
Contributor

@maximearmstrong can you make these changes to the new dlt integration doc as well, if applicable? New docs are going live this Thursday.

@maximearmstrong maximearmstrong force-pushed the maxime/dlt-asset-spec-4 branch from c9a34f8 to 50f3952 Compare January 21, 2025 21:21
@maximearmstrong maximearmstrong force-pushed the maxime/dlt-asset-spec-5 branch from 4e91c66 to 7fb5202 Compare January 21, 2025 21:21
@maximearmstrong maximearmstrong force-pushed the maxime/dlt-asset-spec-4 branch from 50f3952 to ea60ca5 Compare January 21, 2025 22:06
@maximearmstrong maximearmstrong force-pushed the maxime/dlt-asset-spec-5 branch from 7fb5202 to ca5b017 Compare January 21, 2025 22:06
@maximearmstrong maximearmstrong force-pushed the maxime/dlt-asset-spec-4 branch from ea60ca5 to 1ac8e19 Compare January 21, 2025 22:31
@maximearmstrong maximearmstrong force-pushed the maxime/dlt-asset-spec-5 branch from ca5b017 to 3972baf Compare January 21, 2025 22:32
Base automatically changed from maxime/dlt-asset-spec-4 to master January 21, 2025 23:15
@maximearmstrong maximearmstrong force-pushed the maxime/dlt-asset-spec-5 branch from 3972baf to cc8af71 Compare January 21, 2025 23:18
@maximearmstrong maximearmstrong merged commit 7b24512 into master Jan 21, 2025
6 checks passed
@maximearmstrong maximearmstrong deleted the maxime/dlt-asset-spec-5 branch January 21, 2025 23:35
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.

3 participants