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

[daggy-u] include SensorEvaluationContext import in full code example #23029

Merged
merged 1 commit into from
Jul 16, 2024

Conversation

cmpadden
Copy link
Contributor

Summary & Motivation

  • Includes SensorEvaluationContext import in full example at end of "Lesson 9: Building the sensor"

Comment from community member, Joseph:

Hi, I'm going through the Dagster Essentials course and noticed the SensorEvaluationContext import is missing from the last code block of Building the sensor. Just thought y'all should know

@cmpadden cmpadden requested a review from erinkcochran87 July 16, 2024 15:33
@graphite-app graphite-app bot added the area: dagster-university Related to Dagster University label Jul 16, 2024
Copy link

Deploy preview for dagster-docs ready!

Preview available at https://dagster-docs-mg9kdnd69-elementl.vercel.app
https://colton-du-SensorEvaluationContext-import.dagster.dagster-docs.io

Direct link to changed pages:

@erinkcochran87
Copy link
Contributor

Aren't you supposed to be on vacation?!

@cmpadden
Copy link
Contributor Author

Aren't you supposed to be on vacation?!

heh, I'm taking a small break from the mountains of Acadia. 😅

@cmpadden cmpadden merged commit fad3a5c into master Jul 16, 2024
2 checks passed
@cmpadden cmpadden deleted the colton/du-SensorEvaluationContext-import branch July 16, 2024 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: dagster-university Related to Dagster University
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants