You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/07/07 21:40:27 UTC

[GitHub] [airflow] potiuk edited a comment on pull request #16865: Use existing cached_property method to skip pylint error.

potiuk edited a comment on pull request #16865:
URL: https://github.com/apache/airflow/pull/16865#issuecomment-875952706


   Yeah. I see - simply so far we usually solved those kind of problems by bringing-in all missing commits (@ashb  even had a magic git command to get the chain of commits needed). But it has some drawbacks (bringing more than needed sometimes. Just for consideration., I think bringing some refactors/tool commits make it more future-proof and easier to cherry-pick subsequent fixes if needed. I don't mind either, but I think final result is better if we mostly stick to cherry-picking rather than fixing in-place.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org