Skip to content

Dagster Essentials dg #96

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

Conversation

dehume
Copy link
Collaborator

@dehume dehume commented Apr 9, 2025

Convert Dagster Essentials to use dg.

  • Convert project structure to dg
  • Update content to include instructions around dg

Copy link

github-actions bot commented Apr 9, 2025

Deploy preview for dagster-university ready!

✅ Preview
https://dagster-university-o0nli2y46-elementl.vercel.app
https://dennis-ce-976-dagster-essentials-dg-project-structure.dagster-university.dagster-docs.io

Built with commit 68d843c.
This pull request is being automatically deployed with vercel-action

@dehume dehume added the dg Transition to dg label Apr 10, 2025
@dehume dehume changed the base branch from main to dagster-university-dg May 5, 2025 13:15
@dehume dehume marked this pull request as ready for review May 5, 2025 13:15
@dehume dehume requested review from cmpadden and C00ldudeNoonan May 5, 2025 13:18
Copy link
Collaborator

@cmpadden cmpadden left a comment

Choose a reason for hiding this comment

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

The scaffolding is really sweet.

Primary feedback is around venv management, and how that may have changed.

dehume and others added 2 commits May 6, 2025 08:54
Co-authored-by: colton <cmpadden@users.noreply.github.com>
@dehume dehume requested a review from cmpadden May 9, 2025 14:28
@dehume dehume merged commit 2d6e95c into dagster-university-dg May 13, 2025
9 checks passed
@dehume dehume deleted the dennis/ce-976-dagster-essentials-dg-project-structure branch May 13, 2025 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dg Transition to dg
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants