better error when wrong type supplied in deps dictionary
ClosedPublic
Actions

Authored by • sandyryza on Aug 13 2021, 3:18 PM.

Details

Reviewers

• cdecarolis
alangenfeld

Commits

R1:c9cb915168a4: better error when wrong type supplied in deps dictionary

Summary

Came across this when working on Caprisun

Test Plan

Diff Detail

Repository

R1 dagster

Branch

deps-error (branched from master)

Lint

Lint Passed

Unit

No Test Coverage

Event Timeline

• sandyryza created this revision.Aug 13 2021, 3:18 PM

Harbormaster returned this revision to the author for changes because remote builds failed.Aug 13 2021, 3:41 PM

Harbormaster failed remote builds in B34654: Diff 42871!

Harbormaster returned this revision to the author for changes because remote builds failed.Aug 13 2021, 6:39 PM

Harbormaster failed remote builds in B34656: Diff 42874!

Harbormaster completed remote builds in B34664: Diff 42884.Aug 16 2021, 11:46 PM

• sandyryza requested review of this revision.Aug 16 2021, 11:46 PM

alangenfeld accepted this revision.Aug 24 2021, 4:13 PM

This revision is now accepted and ready to land.Aug 24 2021, 4:13 PM

Closed by commit R1:c9cb915168a4: better error when wrong type supplied in deps dictionary (authored by • sandyryza). · Explain WhyAug 24 2021, 4:24 PM

This revision was automatically updated to reflect the committed changes.

• sandyryza added a commit: R1:c9cb915168a4: better error when wrong type supplied in deps dictionary.

Revision Contents
Changeset List

Path

Size

python_modules/

dagster/

core/

definitions/

solid_container.py

4 lines

Diff 42871

View Options

python_modules/dagster/dagster/core/definitions/solid_container.py

better error when wrong type supplied in deps dictionaryClosedPublicActions