Page MenuHomePhabricator

fix missing args in pipeline run roundtrip
ClosedPublic

Authored by alangenfeld on Feb 13 2020, 8:26 PM.

Details

Summary

and improve testing so we dont miss any again

Test Plan

updated tests

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

alangenfeld created this revision.Feb 13 2020, 8:26 PM
alangenfeld updated this revision to Diff 9672.Feb 13 2020, 8:37 PM

future proofing

alangenfeld updated this revision to Diff 9673.Feb 13 2020, 8:58 PM

dont delete to_selector

prha accepted this revision.Feb 13 2020, 9:01 PM
This revision is now accepted and ready to land.Feb 13 2020, 9:01 PM
This revision was automatically updated to reflect the committed changes.