Page MenuHomePhabricator

Use execute_solid
ClosedPublic

Authored by max on Nov 5 2019, 9:46 PM.

Details

Reviewers
alangenfeld
nate
Group Reviewers
Restricted Project
Commits
R1:4bd7ac07d4cb: Use execute_solid
Summary

Don't use one-off shim functions in test where we have blessed facilities.

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 created this revision.Nov 5 2019, 9:46 PM
nate accepted this revision.Nov 5 2019, 9:47 PM
nate added a subscriber: nate.

ha, I didn't even know execute_isolated_solid existed. this is great, thanks for the cleanup

This revision is now accepted and ready to land.Nov 5 2019, 9:47 PM
This revision was landed with ongoing or failed builds.Nov 5 2019, 10:00 PM
Closed by commit R1:4bd7ac07d4cb: Use execute_solid (authored by max). · Explain Why
This revision was automatically updated to reflect the committed changes.