potiuk commented on code in PR #51503: URL: https://github.com/apache/airflow/pull/51503#discussion_r2247751290
########## .gitignore: ########## @@ -117,6 +117,7 @@ celerybeat-schedule .venv* venv* ENV/ +airflow_venv Review Comment: > So if we shouldn't follow this, we should change it or delete it. I can update it in this PR if you'd like. Yes. Please delete it and replace it with simple link to contributing doc. This document is outdated and does not actually properly describe installing airflow for local development. As of 5-6 months or so the only "recommended" way to install airflow for local development is using `uv` (and this is described in the contributing guides) - because airflow repo is now a monorepo with 100+ distributions that are linked through workspace feature. This INSTALLING.md with pyenv is not correct and should be deleted. Please do so. -- 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]
