Skip to content

Testing with Dagster dg #97

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 10, 2025

Convert Testing with Dagster to use dg.

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

There is much less to convert than other courses. Mostly just restructuring to dg project structure and slight updates in content.

@dehume dehume added the dg Transition to dg label Apr 10, 2025
Copy link

github-actions bot commented Apr 10, 2025

Deploy preview for dagster-university ready!

✅ Preview
https://dagster-university-drq4fzwq8-elementl.vercel.app
https://dennis-ce-978-dagster-testing-dg-project-structure.dagster-university.dagster-docs.io

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

@dehume
Copy link
Collaborator Author

dehume commented Apr 10, 2025

Can ignore the "Ensure install options" check. Since we are removing pip install option

@dehume dehume force-pushed the dennis/ce-978-dagster-testing-dg-project-structure branch from fb31eb3 to c545796 Compare April 17, 2025 16:18
@dehume dehume marked this pull request as ready for review April 21, 2025 13:01
@dehume dehume changed the base branch from main to dagster-university-dg May 5, 2025 13:14
@dehume dehume requested review from cmpadden and C00ldudeNoonan May 5, 2025 13:18

To install the python dependencies with [uv](https://docs.astral.sh/uv/).
In order to best use `dg` we will need the Python package manager [`uv`](https://docs.astral.sh/uv/). `uv` will allow us to install `dg` globally and more easily build our virtual environments.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Just for your awareness, we hope to get dg available via homebrew and curl.

I actually already did the homebrew part but it's not being updated automatically via CI, so not recommended yet.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I wonder if we can create a partial to share across courses for this.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yeah. These steps should get reviewed and standardized when we get closer on dg

https://linear.app/dagster-labs/issue/DR-1058/standardize-dagster-u-install-instructions

@dehume dehume merged commit 30e4b06 into dagster-university-dg May 6, 2025
13 checks passed
@dehume dehume deleted the dennis/ce-978-dagster-testing-dg-project-structure branch May 6, 2025 13:42
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