Skip to content

Commit

Permalink
Replace SQL with Common SQL in pre commit (#26058)
Browse files Browse the repository at this point in the history
This pre-commit verify source/target provider for transfer exist.
Now that we have `common.sql` provider there is no need to use SQL along with Common SQL
  • Loading branch information
eladkal authored Aug 30, 2022
1 parent 733d623 commit 8acdc2a
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion airflow/providers/amazon/provider.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -533,7 +533,7 @@ transfers:
target-integration-name: Amazon Simple Storage Service (S3)
how-to-guide: /docs/apache-airflow-providers-amazon/operators/transfer/local_to_s3.rst
python-module: airflow.providers.amazon.aws.transfers.local_to_s3
- source-integration-name: SQL
- source-integration-name: Common SQL
target-integration-name: Amazon Simple Storage Service (S3)
how-to-guide: /docs/apache-airflow-providers-amazon/operators/transfer/sql_to_s3.rst
python-module: airflow.providers.amazon.aws.transfers.sql_to_s3
Expand Down
4 changes: 2 additions & 2 deletions airflow/providers/google/provider.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -817,7 +817,7 @@ transfers:
target-integration-name: Google Cloud Storage (GCS)
how-to-guide: /docs/apache-airflow-providers-google/operators/transfer/trino_to_gcs.rst
python-module: airflow.providers.google.cloud.transfers.trino_to_gcs
- source-integration-name: SQL
- source-integration-name: Common SQL
target-integration-name: Google Cloud Storage (GCS)
python-module: airflow.providers.google.cloud.transfers.sql_to_gcs
- source-integration-name: Google Cloud Storage (GCS)
Expand Down Expand Up @@ -896,7 +896,7 @@ transfers:
target-integration-name: Google Spreadsheet
how-to-guide: /docs/apache-airflow-providers-google/operators/transfer/gcs_to_sheets.rst
python-module: airflow.providers.google.suite.transfers.gcs_to_sheets
- source-integration-name: SQL
- source-integration-name: Common SQL
target-integration-name: Google Spreadsheet
how-to-guide: /docs/apache-airflow-providers-google/operators/transfer/sql_to_sheets.rst
python-module: airflow.providers.google.suite.transfers.sql_to_sheets
Expand Down
2 changes: 1 addition & 1 deletion airflow/providers/slack/provider.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ hooks:
- airflow.providers.slack.hooks.slack_webhook

transfers:
- source-integration-name: SQL
- source-integration-name: Common SQL
target-integration-name: Slack
python-module: airflow.providers.slack.transfers.sql_to_slack
how-to-guide: /docs/apache-airflow-providers-slack/operators/sql_to_slack.rst
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@ def get_all_integration_names(yaml_files) -> List[str]:
all_integrations = [
i['integration-name'] for f in yaml_files.values() if 'integrations' in f for i in f["integrations"]
]
all_integrations += ["SQL", "Local"]
all_integrations += ["Local"]
return all_integrations


Expand Down

0 comments on commit 8acdc2a

Please sign in to comment.