brickflow icon indicating copy to clipboard operation
brickflow copied to clipboard

[BUG] Workflow.health parameter is incorrectly typed

Open nikethrowaway opened this issue 11 months ago • 2 comments

Describe the bug

The type hint for the health parameter on the Workflow class is incorrectly typed. It is typed as Optional[List[JobsHealthRules]]

HOWEVER when you then do a deployment with brickflow of said workflow it will spit out an error, as on the deployment end it expects to find the following type: Optional[JobsHealth]

I'm a nike employee, so had to create a new github account to report this, would appreciate if this is taken care of. Thanks.

nikethrowaway avatar Feb 12 '25 09:02 nikethrowaway

thanks for reporting the bug. will be taken care as part of 1.3.4

pariksheet avatar May 06 '25 09:05 pariksheet

I'll take this one ✋

riccamini avatar May 06 '25 09:05 riccamini