Page MenuHomePhabricator

(dict-where-type-is-used-10) Eliminate wrapping subpackage
ClosedPublic

Authored by schrockn on Thu, Jan 9, 9:16 PM.

Details

Summary

With the previous changes we can eliminate the entire concept
of a "wrapping type system" and instead just talk about remapping python
builtins, which is much easier to understand.

Depends on D1810

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.Thu, Jan 9, 9:16 PM
schrockn updated this revision to Diff 8528.Thu, Jan 9, 9:19 PM
schrockn edited the summary of this revision. (Show Details)
schrockn added reviewers: max, alangenfeld.

upmessage

schrockn updated this revision to Diff 8529.Thu, Jan 9, 9:20 PM

upmessage

This revision is now accepted and ready to land.Thu, Jan 9, 11:00 PM
max accepted this revision.Sat, Jan 11, 12:28 AM

super, see comment

python_modules/dagster/dagster/core/instance/config.py
32

Permissive?

schrockn added inline comments.Sat, Jan 11, 12:36 AM
python_modules/dagster/dagster/core/instance/config.py
32

rename in the next diff