Jarek Potiuk

Results 1573 comments of Jarek Potiuk

> It is probably easiest to tack the linter part on Ruff—it is Rust, but the code to implement a lint rule isn’t that hard if you know Python AST...

Just as a comment to remind in the future - python version in the check_python_version script from #43282 (back-ported in #43310) will also need to be updated before we merge...

Will have to wait for the 2.11 cut-off I guess.

Hey @jscheffl -> I rebased this one on top of the "no pull-request-target" v2-10-test.

> I'll go back to the list with this feedback, but this implementation is tantamount to a parallel scheduler, executor and triggerer implementation so is very unlikely to be accepted...

> Compat tests 2.10.1 failing because python operator has changes related to airflow 3.0.0, should i update back changes to support in 2.10.X? It looks like it's more of "unit...

> ah i got it what you mean by 'assertions work', need to make some adjustments in the python operator tests related to new feature updates. hope that works.. Yes....

> I believe we have to move these scripts as well into standard provider `airflow/utils/python_virtualenv.py` `airflow/utils/python_virtualenv_script.jinja2` ? Yes. Absolutely.

> > > Just one thing about compat provider reuse. > > > > > > yeah am updating imports to use compat. > > I've added the imports to...

And it's not that difficult, most of the code is common in constants, and some other could be replaced with search/replace.