Page MenuHomePhabricator

(new-types-package-org-2) Make builtin mapping stuff a top-level submodule
ClosedPublic

Authored by schrockn on Sat, Jan 11, 1:08 PM.

Details

Summary

Since this orchestrates between dagster.config and soon-to-be
dagster.core.types, but this at the top-level

Depends on D1821

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.Sat, Jan 11, 1:08 PM
schrockn updated this revision to Diff 8617.Sat, Jan 11, 1:27 PM
schrockn retitled this revision from builtins top-level to (new-types-package-org-2) Make builtin mapping stuff a top-level submodule.
schrockn edited the summary of this revision. (Show Details)
schrockn added reviewers: alangenfeld, max.

upmessage

alangenfeld accepted this revision.EditedMon, Jan 13, 4:21 PM

builtins

shims maybe? "builtins" on its own just doesn't communicate much

This revision is now accepted and ready to land.Mon, Jan 13, 4:21 PM

going to land this and then we can bounce around renames