Jens Scheffler
Jens Scheffler
> > I am not sure whether this PR is a good idea. When this is merged you basically break compatibility with Airflow 2.10/2.11 > > Hi @jscheffl , like...
> @eladkal @amoghrajesh @o-nikolas @jscheffl When you get time, could you verify this PR and share your feedback? I am not an expert of the Amazon provider, I'd leave the...
> I might want to extract some of these to separate PRs I'd love some smaller "enablement" PRs to bring the review volume of this down. But I'll start reviewing...
Wohoo! Party!
Except some minor comment I think it is good, let me check what is broken on CI. Nothing related to your code. Ignore that it is "red" atm, will be...
Somthing is wrong with this PR and CI - I don't see the reason but I am very sure it is not related to the changes. I am on it...
> hey @bbovenzi and the rest of the community! > > What do you thing of introducing symbols instead of using colors? I've [created a userscript](https://greasyfork.org/en/scripts/518865-airflow-task-instance-status-enhancer) which does just that,...
I also just noticed it was on the "wrong" branch - but only because I had a test on v2-10-test running that was cancelled due to the merge.
> > I also just noticed it was on the "wrong" branch - but only because I had a test on v2-10-test running that was cancelled due to the merge....
Very cool!