Skip to content

Migrate from dagster-shell to the standard library's supbprocess#58

Open
menzenski wants to merge 3 commits intoquantile-development:masterfrom
menzenski:from-dagster-shell-to-subprocess
Open

Migrate from dagster-shell to the standard library's supbprocess#58
menzenski wants to merge 3 commits intoquantile-development:masterfrom
menzenski:from-dagster-shell-to-subprocess

Conversation

@menzenski
Copy link
Copy Markdown

For #56

This PR is an alternative approach to #57 based on feedback from @alangenfeld here #57 (comment)

@Joseph-A-Pearson
Copy link
Copy Markdown

Hello, I’m checking in to see if this PR is still actively being worked on. I’m needing to upgrade Dagster soon and am wondering if there’s an estimated timeline for merging, or if any help is needed to get it over the line. Thanks in advance!

@menzenski
Copy link
Copy Markdown
Author

@Joseph-A-Pearson I'd be happy to have you help test this. I'm still interested in this (we are similarly stuck on Dagster 1.9 due to this issue). I haven't had much capacity to test it myself, but I'm still hoping to get this resolved.

@prasannarathi
Copy link
Copy Markdown

prasannarathi commented Oct 28, 2025

@menzenski - I am happy to try your PR in a real word if it aligns with @schrockn . There are many Dagster -> Meltano users who would benefit from this.
Please let me know.

@menzenski menzenski marked this pull request as ready for review November 19, 2025 19:53
@menzenski
Copy link
Copy Markdown
Author

@JulesHuisman I've taken this PR out of "draft" status. I see that you've approved it and that you have write access to this repository. Are you able to help get this merged+published?

@JulesHuisman
Copy link
Copy Markdown
Contributor

There are some people who have continued development on the package: https://github.com/camisinc/elt-dagster-ext. As we do not really have a lot of time to keep this package up to date. Would that suffice?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants