Skip to content

Commit

Permalink
[daggy-u] [dbt] - Correct S3 env var key (#23553)
Browse files Browse the repository at this point in the history
## Summary & Motivation

This PR corrects the name of the S3 bucket environment variable in
Lesson 7 to `S3_BUCKET_PREFIX`.

## How I Tested These Changes

Matches the [`constants.py` of the accompanying
project](https://github.com/dagster-io/project-dagster-university/blob/module/dagster-and-dbt/dagster_university/assets/constants.py)
  • Loading branch information
erinkcochran87 authored Aug 20, 2024
1 parent b144c60 commit 9698b0c
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -153,7 +153,7 @@ The following table contains the environment variables we need to create in Dags

---

- `S3_BUCKET_NAME`
- `S3_BUCKET_PREFIX`
- The name of the S3 bucket, by default `"s3://dagster-university/"`, for the S3 bucket where your taxi data will be stored.

{% /table %}
Expand Down

2 comments on commit 9698b0c

@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-5mgu3oni6-elementl.vercel.app

Built with commit 9698b0c.
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-hvcyhnve2-elementl.vercel.app
https://master.dagster.dagster-docs.io

Built with commit 9698b0c.
This pull request is being automatically deployed with vercel-action

Please sign in to comment.