Skip to content

Commit

Permalink
removed yield from link, site is now spam redirect (#25806)
Browse files Browse the repository at this point in the history
## Summary & Motivation
community member [told
us](https://dagster.slack.com/archives/C01U5LFUZJS/p1731056410596729)
that this
[link](https://pythonalgos.com/generator-functions-yield-and-yield-from-in-python/)
explains the difference between yield and yield from redirects to a spam
site.

## How I Tested These Changes

## Changelog
I removed the link in the course.
  • Loading branch information
C00ldudeNoonan authored Nov 8, 2024
1 parent 0902aac commit 04d4f98
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ We’ll only create one `@dbt_assets` definition for now, but in a later lesson,
- We use the `dbt` argument (which is a `DbtCliResource`) to execute a dbt command through its `.cli` method.
- The `.stream()` method fetches the events and results of this dbt execution.
- This is one of multiple ways to get the Dagster events, such as what models materialized or tests passed. We recommend starting with this and exploring other methods in the future as your use cases grow (such as fetching the run artifacts after a run). In this case, the above line will execute `dbt run`.
- The results of the `stream` are a Python generator of what Dagster events happened. We used [`yield from`](https://pythonalgos.com/generator-functions-yield-and-yield-from-in-python/) (not just `yield`!) to have Dagster track asset materializations.
- The results of the `stream` are a Python generator of what Dagster events happened. We used `yield from` (not just `yield`!) to have Dagster track asset materializations.

At this point, `dbt.py` should look like this:

Expand Down

2 comments on commit 04d4f98

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Deploy preview for dagster-university ready!

✅ Preview
https://dagster-university-8616einqi-elementl.vercel.app

Built with commit 04d4f98.
This pull request is being automatically deployed with vercel-action

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Deploy preview for dagster-docs ready!

✅ Preview
https://dagster-docs-kwcmuvkss-elementl.vercel.app
https://master.dagster.dagster-docs.io

Built with commit 04d4f98.
This pull request is being automatically deployed with vercel-action

Please sign in to comment.