Page MenuHomePhabricator

(dict-where-type-is-used-6) Rename config list to array in graphql and dagit
ClosedPublic

Authored by schrockn on Thu, Jan 9, 2:43 PM.

Details

Summary

Flow this change down to dagit. Also removed isList and
isNullable since it was unused on the client

Depends on D1804

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, 2:43 PM
schrockn updated this revision to Diff 8501.Thu, Jan 9, 2:50 PM
schrockn edited the summary of this revision. (Show Details)

upmessage

schrockn updated this revision to Diff 8559.Thu, Jan 9, 11:23 PM
schrockn added reviewers: alangenfeld, max.

upmessage

alangenfeld accepted this revision.Thu, Jan 9, 11:25 PM

test plan
BK

dagit test coverage is weak so I would do a quick sanity run

This revision is now accepted and ready to land.Thu, Jan 9, 11:25 PM