-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Add code for a use-case repository #23460
Conversation
Deploy preview for dagster-university ready! ✅ Preview Built with commit 8745130. |
8745130
to
eb066f9
Compare
e290f04
to
1583412
Compare
examples/use_case_repository/use_case_repository/guides/snowflake_to_s3_embedded_elt.md
Show resolved
Hide resolved
examples/use_case_repository/use_case_repository/guides/pipes_cli_command.md
Outdated
Show resolved
Hide resolved
examples/use_case_repository/use_case_repository/guides/pipes_cli_command.md
Show resolved
Hide resolved
examples/use_case_repository/use_case_repository/guides/pipes_cli_command.md
Show resolved
Hide resolved
examples/use_case_repository/use_case_repository/guides/pipes_cli_command.md
Outdated
Show resolved
Hide resolved
examples/use_case_repository/use_case_repository/guides/pipes_cli_command.md
Outdated
Show resolved
Hide resolved
examples/use_case_repository/use_case_repository/guides/pipes_cli_command.md
Outdated
Show resolved
Hide resolved
examples/use_case_repository/use_case_repository/guides/pipes_cli_command.md
Outdated
Show resolved
Hide resolved
examples/use_case_repository/use_case_repository/guides/snowflake_to_s3_embedded_elt.md
Outdated
Show resolved
Hide resolved
examples/use_case_repository/use_case_repository/guides/snowflake_to_s3_embedded_elt.md
Outdated
Show resolved
Hide resolved
examples/use_case_repository/use_case_repository/guides/pipes_cli_command.md
Outdated
Show resolved
Hide resolved
examples/use_case_repository/use_case_repository/guides/pipes_cli_command.md
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Back to you!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I left a few comments about the template, but they're really minor. I'm good with this first pass, but may have more feedback once I can see a live version. :)
Summary & Motivation
This adds the
use_case_repository
package under examples which will be used bydagster-website
to populate the the Repository on our website.All
.md
and.py
files inuse_case_repository/guides
from the master branch will be loaded to our website during the build process.A local server is provided in order to quickly view your markdown files using
make webserver
. Refer to theREADME.md
for more information.How I Tested These Changes
local testing