Bugra Ozturk
Bugra Ozturk
>We only want to count its immediate children and their states. So section_2, should only have a count of 2 and states of failed: 1, success: 1 I misunderstood your...
I fixed the mypy check, I am surprised how my local pre-commit couldn't catch that. Additionally, changed the `child_state`, I agree, it can reduce confusion
>* Can you check that limit is actually being respected? In my manual testing, I was trying to only fetch 14 runs but I always got the default of 100....
Amazing, thanks both of you for the open-minded approach and constructive feedback! :) I really appreciate the great work done here. Handling recursive logic like this isn’t easy. This work...
>Can you check that limit is actually being respected? In my manual testing, I was trying to only fetch 14 runs but I always got the default of 100. @pierrejeambrun...
I double-checked the parameters created with `filter_param_factory`, and they weren’t working as expected. 😕 To address this, I needed to adjust the `run_types` and `run_states` parameter names to `run_type` and...
> Code and test cases look good, beside the couple of nits, ready to merge. Amazing news, just pushed the changes and resolved the threads. Thanks a lot for your...
Mostly documentation updates. My changes looks good
Thanks Ash for the review! I will address them soon
Same here! I would suggest to try `airflowctl` too to leverage API through automation and would be great to see what is needed more while we solidify the foundations :)