Jarek Potiuk
Jarek Potiuk
Nice deep dive. Sounds plausible. A proof of concept quick fix/PR would be great :)
Indeed, I think it would be great to just document the different scenarios how dags.test could be used. This would be a fantastic contribution to explain some of the tests...
And if it comes from someone who actually **finds it useful** and **can describe it in the way easy to grasp by similar people** - that's doubly powerful
> I'm glad to volunteer to extend Airflow System Test and `dag.test()` documentation, in case it's an effort that may be welcome. > > (While the basics were very well...
Not sure why you are calling me - that's not my area of expertise. Generally what we prefer is when you are pinging to attract maintainer to review - ping...
> While I understand your point, please note that the auto-assigned reviewers haven't reviewed any PRs in the last 6 months. I tagged you because the standard process didn't seem...
This is really nice and clean and it actually shows why we actually **should** handle async tasks this way - @dstandish -> this PR is now extracted by @dabla to...
@jroachgolf84 -> any more comments ?
I recommend you to install and use `prek` - as described in our docs. It will automatically fix all such problems when you commit stuff.
And the problem is that you have some extra spaces at the end that you don't see