potiuk commented on PR #28578:
URL: https://github.com/apache/airflow/pull/28578#issuecomment-1364539136

   BTW. Rich is a different problem - it is >= limited to force an upgrade for 
everyone for a buggy version, but we do not want to specifically synchronize it 
to specific version. 
   
   There is a big difference - rich is just to "print" output to the user (and 
auto-update of rich is something that happens automatically with `pre-comit 
auto-update`. And we REALLY WANT to keep it syncrhonized in pre-commits because 
black reformats our generated code, so we want to make sure the reformatting is 
not conflicting with regular formatting applied by the pre-commit. If black 
changes the decision about formatting (happened in the past) this would mean 
that our pre-commits will be cancelling each-others formatting when a new file 
is generated (that's why also we want to fix black so that formatting is the 
same for users.


-- 
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