potiuk commented on issue #41641:
URL: https://github.com/apache/airflow/issues/41641#issuecomment-2484421421

   > 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 and just a 
general idea about C-like languages. The rewrite part is a lot more difficult, 
so it’s probably better to implement this as a different tool in Python with 
libcst. I’m thinking something like
   
   Actually I am convinced too - I quite like this one after a bit of thought. 
This is not something that might be maintained by a lot of people and a number 
of contributors, and even for them, this is so far from the main "airflow code" 
- it's really a "one-time" tool - that it might be worth treating it as our 
first "rust experiment". And I quite agree that, the AST code on it's own is 
not really that "pythonic" and if you know what you want, and have already 
existing examples, adding a new rule in RUST, should not be difficult even if 
you do not know it (and AI driven development here might be even pretty cool 
exercise). I'd myself be happy to add a few rules at some point of time and 
maybe even take part in implementing the tooling for rust for our CI 
environment.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to