Page MenuHomePhabricator

Fix bug in config post-processing
ClosedPublic

Authored by nate on Fri, Feb 14, 10:40 PM.

Details

Summary

typo - result has no "error" property, its "errors" which is a list of errors that needs to be flattened.

Hit this in Helm chart testing

Test Plan

unit

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

nate created this revision.Fri, Feb 14, 10:40 PM
nate edited the summary of this revision. (Show Details)Fri, Feb 14, 10:42 PM
nate added reviewers: alangenfeld, max.
max added a comment.Fri, Feb 14, 10:47 PM

Can we add a test also!

alangenfeld accepted this revision.Fri, Feb 14, 10:48 PM

nice good catch

should be easy to sneak a test in to python_modules/dagster/dagster_tests/core_tests/config_types_tests/test_source_types.py

This revision is now accepted and ready to land.Fri, Feb 14, 10:48 PM
This revision was automatically updated to reflect the committed changes.