Page MenuHomePhabricator

(new-types-package-org-4) Make name more clear. Eliminate named dicts and selectors
ClosedPublic

Authored by schrockn on Jan 12 2020, 5:53 PM.

Details

Summary

First just eliminate the API to specify name on a Selector or
a Shape.

Then name to "given_name". It is only valid for scalars and enums. No other types are given names.

Depends on D1823

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 retitled this revision from Killing config name to (new-types-package-org-4) Make name more clear. Eliminate named dicts and selectors.
schrockn edited the summary of this revision. (Show Details)
schrockn added reviewers: alangenfeld, max.

upmessage

alangenfeld added inline comments.
python_modules/dagster/dagster/config/field_utils.py
57–67

byebye

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