Skip to content

Commit

Permalink
Remove validate-operators-init from v2-* static check branches (apach…
Browse files Browse the repository at this point in the history
…e#37238)

In v2-7* branches we usually don't cherry-pick provider changes and
we generally skip all the provider-related static checks and tests.
  • Loading branch information
potiuk authored Feb 8, 2024
1 parent e27f76e commit 53185ab
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 10 deletions.
1 change: 1 addition & 0 deletions dev/breeze/src/airflow_breeze/utils/selective_checks.py
Original file line number Diff line number Diff line change
Expand Up @@ -854,6 +854,7 @@ def skip_pre_commits(self) -> str:
"check-extra-packages-references",
"check-provider-yaml-valid",
"lint-helm-chart",
"validate-operators-init",
)
)

Expand Down
12 changes: 2 additions & 10 deletions dev/breeze/tests/test_selective_checks.py
Original file line number Diff line number Diff line change
Expand Up @@ -698,7 +698,7 @@ def test_expected_output_pull_request_main(
"docs-build": "true",
"docs-list-as-string": "apache-airflow docker-stack",
"full-tests-needed": "true",
"skip-pre-commits": "check-airflow-provider-compatibility,check-extra-packages-references,check-provider-yaml-valid,identity,lint-helm-chart,mypy-core,mypy-dev,mypy-docs,mypy-providers",
"skip-pre-commits": "check-airflow-provider-compatibility,check-extra-packages-references,check-provider-yaml-valid,identity,lint-helm-chart,mypy-core,mypy-dev,mypy-docs,mypy-providers,validate-operators-init",
"skip-provider-tests": "true",
"upgrade-to-newer-dependencies": "false",
"parallel-test-types-list-as-string": "API Always BranchExternalPython "
Expand Down Expand Up @@ -744,9 +744,6 @@ def test_expected_output_full_tests_needed(
"docs-build": "false",
"docs-list-as-string": None,
"full-tests-needed": "false",
"skip-pre-commits": "check-airflow-provider-compatibility,check-extra-packages-references,"
"check-provider-yaml-valid,flynt,identity,lint-helm-chart,"
"mypy-core,mypy-dev,mypy-docs,mypy-providers,ts-compile-format-lint-www",
"upgrade-to-newer-dependencies": "false",
"skip-provider-tests": "true",
"parallel-test-types-list-as-string": None,
Expand All @@ -773,9 +770,6 @@ def test_expected_output_full_tests_needed(
"docs-build": "true",
"docs-list-as-string": "apache-airflow docker-stack",
"full-tests-needed": "false",
"skip-pre-commits": "check-airflow-provider-compatibility,check-extra-packages-references,"
"check-provider-yaml-valid,identity,lint-helm-chart,"
"mypy-core,mypy-dev,mypy-docs,mypy-providers,ts-compile-format-lint-www",
"run-kubernetes-tests": "true",
"upgrade-to-newer-dependencies": "false",
"skip-provider-tests": "true",
Expand Down Expand Up @@ -806,7 +800,6 @@ def test_expected_output_full_tests_needed(
"docs-build": "true",
"docs-list-as-string": "apache-airflow docker-stack",
"full-tests-needed": "false",
"skip-pre-commits": "check-airflow-provider-compatibility,check-extra-packages-references,check-provider-yaml-valid,identity,lint-helm-chart,mypy-core,mypy-dev,mypy-docs,mypy-providers,ts-compile-format-lint-www",
"run-kubernetes-tests": "true",
"upgrade-to-newer-dependencies": "false",
"skip-provider-tests": "true",
Expand Down Expand Up @@ -835,7 +828,6 @@ def test_expected_output_full_tests_needed(
"run-kubernetes-tests": "false",
"upgrade-to-newer-dependencies": "false",
"skip-provider-tests": "true",
"skip-pre-commits": "check-airflow-provider-compatibility,check-extra-packages-references,check-provider-yaml-valid,identity,lint-helm-chart,mypy-core,mypy-dev,mypy-docs,mypy-providers,ts-compile-format-lint-www",
"parallel-test-types-list-as-string": "API Always BranchExternalPython BranchPythonVenv "
"CLI Core ExternalPython Operators Other PlainAsserts PythonVenv Serialization WWW",
"needs-mypy": "true",
Expand Down Expand Up @@ -1066,7 +1058,7 @@ def test_expected_output_pull_request_target(
"needs-helm-tests": "false",
"run-tests": "true",
"docs-build": "true",
"skip-pre-commits": "check-airflow-provider-compatibility,check-extra-packages-references,check-provider-yaml-valid,identity,lint-helm-chart,mypy-core,mypy-dev,mypy-docs,mypy-providers",
"skip-pre-commits": "check-airflow-provider-compatibility,check-extra-packages-references,check-provider-yaml-valid,identity,lint-helm-chart,mypy-core,mypy-dev,mypy-docs,mypy-providers,validate-operators-init",
"docs-list-as-string": "apache-airflow docker-stack",
"upgrade-to-newer-dependencies": "true",
"parallel-test-types-list-as-string": "API Always BranchExternalPython BranchPythonVenv "
Expand Down

0 comments on commit 53185ab

Please sign in to comment.