dagster icon indicating copy to clipboard operation
dagster copied to clipboard

Clarify properties on Definitions

Open dpeng817 opened this issue 8 months ago • 6 comments

The assets property on definition did something very different from get_assets_def property, and I found that very confusing. The only reason we needed public properties for the "dagster defs" within a definitions object was for Definitions.merge. Instead, let's just consolidate the original args into a typeddict that we can access easily, and makes it clear what we're retrieving.

dpeng817 avatar Jun 25 '24 21:06 dpeng817