Re: [PR] [FIX] Fixed databricks repair run deferrable [airflow]
Lee-W merged PR #44213: URL: https://github.com/apache/airflow/pull/44213 -- 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-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] [FIX] Fixed databricks repair run deferrable [airflow]
boring-cyborg[bot] commented on PR #44213: URL: https://github.com/apache/airflow/pull/44213#issuecomment-2534986043 Awesome work, congrats on your first merged pull request! You are invited to check our [Issue Tracker](https://github.com/apache/airflow/issues) for additional contributions. -- 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-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] [FIX] Fixed databricks repair run deferrable [airflow]
raghvendra-singh1 commented on PR #44213: URL: https://github.com/apache/airflow/pull/44213#issuecomment-2533755161 @Lee-W Passed the CI tests. Could you please review now? -- 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-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] [FIX] Fixed databricks repair run deferrable [airflow]
Lee-W commented on PR #44213: URL: https://github.com/apache/airflow/pull/44213#issuecomment-2533646651 rebased from the latest main to see whether the last test passes -- 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-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] [FIX] Fixed databricks repair run deferrable [airflow]
raghvendra-singh1 commented on PR #44213: URL: https://github.com/apache/airflow/pull/44213#issuecomment-2532671110 @Lee-W Made some changes. Those failing CI tests should be fixed now. Could you please review? -- 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-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] [FIX] Fixed databricks repair run deferrable [airflow]
Lee-W commented on PR #44213: URL: https://github.com/apache/airflow/pull/44213#issuecomment-2527112601 > @Lee-W can you please review and merge this bug fix PR Sure, will take a look once the test and CI pass -- 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-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] [FIX] Fixed databricks repair run deferrable [airflow]
gaurav7261 commented on PR #44213: URL: https://github.com/apache/airflow/pull/44213#issuecomment-2527095542 @Lee-W can you please review and merge this bug fix PR -- 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-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] [FIX] Fixed databricks repair run deferrable [airflow]
raghvendra-singh1 commented on PR #44213: URL: https://github.com/apache/airflow/pull/44213#issuecomment-2493191338 Sure, will look into that -- 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-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] [FIX] Fixed databricks repair run deferrable [airflow]
Lee-W commented on PR #44213: URL: https://github.com/apache/airflow/pull/44213#issuecomment-2490194490 Will need to get the tests fixed -- 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-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] [FIX] Fixed databricks repair run deferrable [airflow]
boring-cyborg[bot] commented on PR #44213: URL: https://github.com/apache/airflow/pull/44213#issuecomment-2488739209 Congratulations on your first Pull Request and welcome to the Apache Airflow community! If you have any issues or are unsure about any anything please check our Contributors' Guide (https://github.com/apache/airflow/blob/main/contributing-docs/README.rst) Here are some useful points: - Pay attention to the quality of your code (ruff, mypy and type annotations). Our [pre-commits]( https://github.com/apache/airflow/blob/main/contributing-docs/08_static_code_checks.rst#prerequisites-for-pre-commit-hooks) will help you with that. - In case of a new feature add useful documentation (in docstrings or in `docs/` directory). Adding a new operator? Check this short [guide](https://github.com/apache/airflow/blob/main/docs/apache-airflow/howto/custom-operator.rst) Consider adding an example DAG that shows how users should use it. - Consider using [Breeze environment](https://github.com/apache/airflow/blob/main/dev/breeze/doc/README.rst) for testing locally, it's a heavy docker but it ships with a working Airflow and a lot of integrations. - Be patient and persistent. It might take some time to get a review or get the final approval from Committers. - Please follow [ASF Code of Conduct](https://www.apache.org/foundation/policies/conduct) for all communication including (but not limited to) comments on Pull Requests, Mailing list and Slack. - Be sure to read the [Airflow Coding style]( https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#coding-style-and-best-practices). - Always keep your Pull Requests rebased, otherwise your build might fail due to changes not related to your commits. Apache Airflow is a community-driven project and together we are making it better 🚀. In case of doubts contact the developers at: Mailing List: d...@airflow.apache.org Slack: https://s.apache.org/airflow-slack -- 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-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org