Page MenuHomePhabricator

(type-system-changes-4) Remove Anonymous Registry
ClosedPublic

Authored by schrockn on Jan 27 2020, 9:52 PM.

Details

Summary

As agreed, this is problematic given possibility of
import-order-driven behavior change and other issues

Depends on D1895

Test Plan

BK

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

schrockn created this revision.Jan 27 2020, 9:52 PM
schrockn updated this revision to Diff 9003.Jan 28 2020, 12:46 AM
schrockn edited the summary of this revision. (Show Details)
schrockn added reviewers: max, alangenfeld.

upmessage

schrockn updated this revision to Diff 9072.Jan 29 2020, 1:40 AM

upmessage

max accepted this revision.Feb 4 2020, 11:16 PM

We will need to make corresponding changes to the tutorial

This revision is now accepted and ready to land.Feb 4 2020, 11:16 PM

I actually don't see where this behavior is needed for the tutorials

This revision was automatically updated to reflect the committed changes.