Page MenuHomePhabricator

Modernize dagstermill test setup
ClosedPublic

Authored by max on Mon, Jan 4, 8:32 PM.

Details

Summary

Use contemporary APIs in test

Test Plan

Unit

Diff Detail

Repository
R1 dagster
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

max requested review of this revision.Mon, Jan 4, 8:54 PM

cleanup

python_modules/libraries/dagstermill/dagstermill/serialize.py
4–40

i forgot about /never saw this spooky zone - out of scope for this diff but we should figure out the Right Way (tm) to do this

This revision is now accepted and ready to land.Tue, Jan 5, 8:07 PM
python_modules/libraries/dagstermill/dagstermill/serialize.py
4–40

next diff!

This revision was automatically updated to reflect the committed changes.