Page MenuHomePhabricator

(new-types-package-org-1) dagster.core.types.config --> dagster.config
ClosedPublic

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

Details

Summary

This PR makes our config system a top-level subpackage
and a peer to check. This makes the layering much more clear. Eventually
I want to never talk about "config types" and "runtime types" ever
again. Instead we'll have the "config system" (which could end up
being its own standalone module) and the "dagster types". We shall see
though.

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:04 PM
schrockn updated this revision to Diff 8616.Sat, Jan 11, 1:24 PM
schrockn retitled this revision from config to top-level subpackage to (new-types-package-org-1) dagster.core.types.config --> dagster.config.
schrockn edited the summary of this revision. (Show Details)
schrockn added reviewers: max, alangenfeld.

upmessage

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