sjyangkevin commented on code in PR #51503: URL: https://github.com/apache/airflow/pull/51503#discussion_r2138826297
########## .gitignore: ########## @@ -117,6 +117,7 @@ celerybeat-schedule .venv* venv* ENV/ +airflow_venv Review Comment: We have multiple ways to set up airflow, and I believe above is one of the options. I first found the QuickStart is very helpful https://github.com/apache/airflow/blob/main/contributing-docs/03_contributors_quick_start.rst. If you want to know more details about the setup, the Breeze documentation is also very helpful. https://github.com/apache/airflow/blob/main/dev/breeze/doc/README.rst. I find it is very easy to setup clean environments even when you need to switch between airflow versions, as well as the test suites , pre-commit commands can help you run the same tests as the ones run in CI. I believe this can be helpful to get your PR to a ready stage. -- 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]
