I was browsing the repo for good first issues as a new contributor, and found #16802 interesting. It looks like @sam-goodwin's PR #20979 is nearly complete, except for a change needed for one of the imports. I used that PR's solution, but also resolved the import issue.
How I Tested These Changes
I am currently running python -m pytest python_modules/dagster/dagster_tests to run the tests locally, though it is taking some time to complete. The tests appear to be passing so far!
Summary & Motivation
I was browsing the repo for good first issues as a new contributor, and found #16802 interesting. It looks like @sam-goodwin's PR #20979 is nearly complete, except for a change needed for one of the imports. I used that PR's solution, but also resolved the import issue.
How I Tested These Changes
I am currently running
python -m pytest python_modules/dagster/dagster_teststo run the tests locally, though it is taking some time to complete. The tests appear to be passing so far!