allenc97 commented on code in PR #69509:
URL: https://github.com/apache/airflow/pull/69509#discussion_r3597661373
##########
dev/breeze/tests/test_selective_checks.py:
##########
@@ -1083,7 +1083,7 @@ def assert_outputs_are_printed(expected_outputs:
dict[str, str], stderr: str):
("providers/amazon/src/airflow/providers/amazon/provider.yaml",),
{
"selected-providers-list-as-string": "amazon apache.hive
cncf.kubernetes "
- "common.compat common.messaging common.sql exasol ftp google
http imap microsoft.azure "
+ "common.compat common.messaging common.sql databricks exasol
ftp google http imap microsoft.azure "
Review Comment:
I took a closer look at this - it seems as of
https://github.com/apache/airflow/issues/44511 the intention is to no longer
rely on provider.yaml for dependency management, e.g quoting `This means that
dependencies should be moved from providrer.yaml files to pyproject.toml`.
This is perhaps also confusing because docs in various places are incorrect,
e.g https://github.com/apache/airflow/blob/main/generated/README.md still says
to modify provider.yaml, but no such section exists in say,
https://github.com/apache/airflow/blob/providers-databricks/7.8.3/providers/amazon/provider.yaml.
--
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]
