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

[deprecate-source-assets] remove SourceAsset from assorted docs #23718

Merged
merged 1 commit into from
Aug 21, 2024

Conversation

sryza
Copy link
Contributor

@sryza sryza commented Aug 16, 2024

Summary & Motivation

SourceAsset is a deprecated concept, but our dlt docs and some random guides still reference them. This PR switches mentions to AssetSpec and "external asset".

How I Tested These Changes

@sryza sryza requested a review from yuhan August 16, 2024 21:53
@graphite-app graphite-app bot added the area: docs Related to documentation in general label Aug 16, 2024
@graphite-app graphite-app bot requested a review from erinkcochran87 August 16, 2024 21:53
@sryza sryza force-pushed the random-source-asset-docs branch from 6f2f600 to 01c596f Compare August 21, 2024 18:38
@sryza sryza force-pushed the random-source-asset-docs branch from 01c596f to 08d6ae8 Compare August 21, 2024 22:02
@sryza sryza merged commit aff5ad4 into master Aug 21, 2024
3 checks passed
@sryza sryza deleted the random-source-asset-docs branch August 21, 2024 22:15
sryza added a commit that referenced this pull request Aug 24, 2024
## Summary & Motivation

SourceAsset is a deprecated concept, but our dlt docs and some random
guides still reference them. This PR switches mentions to AssetSpec and
"external asset".

## How I Tested These Changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: docs Related to documentation in general
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants