This is an automated email from the ASF dual-hosted git repository.
dstandish pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new a66f95539a Reduce max length for pre-commit hook names (#31517)
a66f95539a is described below
commit a66f95539a420ef91f145124c7e8a5394f535c61
Author: Daniel Standish <[email protected]>
AuthorDate: Wed May 24 11:21:52 2023 -0700
Reduce max length for pre-commit hook names (#31517)
The current version of macbook 16 loses a few "effective" pixels
horizontally, so we need to reduce hook name length in order to keep each
message on one line.
---
.pre-commit-config.yaml | 18 +-
STATIC_CODE_CHECKS.rst | 428 ++++++++++++++++++++++++------------------------
2 files changed, 223 insertions(+), 223 deletions(-)
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index ac807a2821..0430658d17 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -212,7 +212,7 @@ repos:
- id: debug-statements
name: Detect accidentally committed debug statements
- id: check-builtin-literals
- name: Require literal syntax when initializing Python builtin types
+ name: Require literal syntax when initializing builtin types
exclude: ^.*/.*_vendor/
- id: detect-private-key
name: Detect if private key is added to the repository
@@ -402,7 +402,7 @@ repos:
pass_filenames: true
files: \.py$
- id: check-links-to-example-dags-do-not-use-hardcoded-versions
- name: Check that example dags do not use hard-coded version numbers
+ name: Verify example dags do not use hard-coded version numbers
description: The links to example dags should use |version| as version
specification
language: pygrep
entry: >
@@ -569,7 +569,7 @@ repos:
files: ^airflow/.*\.py$
- id: check-core-deprecation-classes
language: pygrep
- name: Verify using of dedicated Airflow deprecation classes in core
+ name: Verify usage of Airflow deprecation classes in core
entry: category=DeprecationWarning|category=PendingDeprecationWarning
files: \.py$
exclude:
^airflow/configuration\.py$|^airflow/providers|^scripts/in_container/verify_providers\.py$
@@ -599,7 +599,7 @@ repos:
pass_filenames: true
- id: check-start-date-not-used-in-defaults
language: pygrep
- name: "'start_date' not to be defined in default_args in example_dags"
+ name: start_date not to be defined in default_args in example_dags
entry:
"default_args\\s*=\\s*{\\s*(\"|')start_date(\"|')|(\"|')start_date(\"|'):"
files: \.*example_dags.*\.py$
exclude: ^.*/.*_vendor/
@@ -619,7 +619,7 @@ repos:
additional_dependencies: ['click', 'rich>=12.4.4', 'pyyaml']
require_serial: true
- id: check-airflow-config-yaml-consistent
- name: Checks for consistency between config.yml and default_config.cfg
+ name: Check consistency between config.yml and default_config.cfg
language: python
entry: ./scripts/ci/pre_commit/pre_commit_yaml_to_cfg.py
files: config\.yml$|default_airflow\.cfg$|default\.cfg$
@@ -707,10 +707,10 @@ repos:
language: python
require_serial: true
- id: check-pre-commit-information-consistent
- name: Update information re pre-commit hooks and verify ids and names
+ name: Validate hook IDs & names and sync with docs
entry: ./scripts/ci/pre_commit/pre_commit_check_pre_commit_hooks.py
args:
- - --max-length=64
+ - --max-length=60
language: python
files:
^\.pre-commit-config\.yaml$|^scripts/ci/pre_commit/pre_commit_check_pre_commit_hook_names\.py$
additional_dependencies: ['pyyaml', 'jinja2', 'black==22.12.0',
'tabulate', 'rich>=12.4.4']
@@ -817,7 +817,7 @@ repos:
require_serial: true
additional_dependencies: ['jsonschema>=3.2.0,<5.0', 'PyYAML==5.3.1',
'requests==2.25.0']
- id: lint-json-schema
- name: Lint airflow/config_templates/config.yml file with JSON Schema
+ name: Lint config_templates/config.yml file with JSON Schema
entry: ./scripts/ci/pre_commit/pre_commit_json_schema.py
args:
- --spec-file
@@ -916,7 +916,7 @@ repos:
pass_filenames: false
files: ^tests/.*\.py$
- id: ts-compile-format-lint-www
- name: TS types generation and ESLint/Prettier against current UI files
+ name: TS types generation / ESLint / Prettier against UI files
language: node
'types_or': [javascript, ts, tsx, yaml, css, json]
files: ^airflow/www/static/js/|^airflow/api_connexion/openapi/v1\.yaml$
diff --git a/STATIC_CODE_CHECKS.rst b/STATIC_CODE_CHECKS.rst
index 97e1552abf..c7bb2528a4 100644
--- a/STATIC_CODE_CHECKS.rst
+++ b/STATIC_CODE_CHECKS.rst
@@ -137,220 +137,220 @@ require Breeze Docker image to be built locally.
.. BEGIN AUTO-GENERATED STATIC CHECK LIST
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| ID | Description
| Image |
-+===========================================================+==================================================================+=========+
-| black | Run black
(Python formatter) | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| blacken-docs | Run black on
Python code blocks in documentation files | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-aiobotocore-optional | Check if
aiobotocore is an optional dependency only | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-airflow-config-yaml-consistent | Checks for
consistency between config.yml and default_config.cfg | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-airflow-provider-compatibility | Check
compatibility of Providers with Airflow | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-apache-license-rat | Check if
licenses are OK for Apache | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-base-operator-partial-arguments | Check
BaseOperator and partial() arguments | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-base-operator-usage | * Check
BaseOperator[Link] core imports | |
-| | * Check
BaseOperator[Link] other imports | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-boring-cyborg-configuration | Checks for
Boring Cyborg configuration consistency | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-breeze-top-dependencies-limited | Breeze should
have small number of top-level dependencies | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-builtin-literals | Require literal
syntax when initializing Python builtin types | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-changelog-has-no-duplicates | Check changelogs
for duplicate entries | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-core-deprecation-classes | Verify using of
dedicated Airflow deprecation classes in core | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-daysago-import-from-utils | Make sure
days_ago is imported from airflow.utils.dates | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-decorated-operator-implements-custom-name | Check @task
decorator implements custom_operator_name | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-docstring-param-types | Check that
docstrings do not specify param types | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-example-dags-urls | Check that
example dags url include provider versions | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-executables-have-shebangs | Check that
executables have shebang | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-extra-packages-references | Checks setup
extra packages | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-extras-order | Check order of
extras in Dockerfile | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-for-inclusive-language | Check for
language that we do not accept as community | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-hooks-apply | Check if all
hooks apply to the repository | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-incorrect-use-of-LoggingMixin | Make sure
LoggingMixin is not used alone | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-init-decorator-arguments | Check model
__init__ and decorator arguments are in sync | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-lazy-logging | Check that all
logging methods are lazy | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-links-to-example-dags-do-not-use-hardcoded-versions | Check that
example dags do not use hard-coded version numbers | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-merge-conflict | Check that merge
conflicts are not being committed | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-newsfragments-are-valid | Check
newsfragments are valid | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-no-airflow-deprecation-in-providers | Do not use
DeprecationWarning in providers | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-no-providers-in-core-examples | No providers
imports in core example DAGs | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-no-relative-imports | No relative
imports | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-only-new-session-with-provide-session | Check
NEW_SESSION is only used with @provide_session | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-persist-credentials-disabled-in-github-workflows | Check that
workflow files have persist-credentials disabled | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-pre-commit-information-consistent | Update
information re pre-commit hooks and verify ids and names | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-provide-create-sessions-imports | Check
provide_session and create_session imports | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-provider-yaml-valid | Validate
provider.yaml files | * |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-providers-init-file-missing | Provider init
file is missing | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-providers-subpackages-init-file-exist | Provider
subpackage init files are there | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-pydevd-left-in-code | Check for pydevd
debug statements accidentally left | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-revision-heads-map | Check that the
REVISION_HEADS_MAP is up-to-date | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-safe-filter-usage-in-html | Don't use safe
in templates | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-setup-order | Check order of
dependencies in setup.cfg and setup.py | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-start-date-not-used-in-defaults | 'start_date' not
to be defined in default_args in example_dags | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-system-tests-present | Check if system
tests have required segments of code | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-system-tests-tocs | Check that
system tests is properly added | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-tests-unittest-testcase | Check that unit
tests do not inherit from unittest.TestCase | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-urlparse-usage-in-code | Don't use
urlparse in code | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| check-xml | Check XML files
with xmllint | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| codespell | Run codespell to
check for common misspellings in files | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| compile-www-assets | Compile www
assets | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| compile-www-assets-dev | Compile www
assets in dev mode | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| create-missing-init-py-files-tests | Create missing
init.py files in tests | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| debug-statements | Detect
accidentally committed debug statements | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| detect-private-key | Detect if
private key is added to the repository | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| doctoc | Add TOC for
Markdown and RST files | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| end-of-file-fixer | Make sure that
there is an empty line at the end | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| fix-encoding-pragma | Remove encoding
header from Python files | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| flynt | Run flynt string
format converter for Python | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| identity | Print input to
the static check hooks for troubleshooting | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| insert-license | * Add license
for all SQL files | |
-| | * Add license
for all RST files | |
-| | * Add license
for all CSS/JS/JSX/PUML/TS/TSX files | |
-| | * Add license
for all JINJA template files | |
-| | * Add license
for all Shell files | |
-| | * Add license
for all Python files | |
-| | * Add license
for all XML files | |
-| | * Add license
for all Helm template files | |
-| | * Add license
for all YAML files except Helm templates | |
-| | * Add license
for all Markdown files | |
-| | * Add license
for all other files | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| lint-chart-schema | Lint
chart/values.schema.json file | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| lint-css | stylelint
| |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| lint-dockerfile | Lint Dockerfile
| |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| lint-helm-chart | Lint Helm Chart
| |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| lint-json-schema | * Lint JSON
Schema files with JSON Schema | |
-| | * Lint NodePort
Service with JSON Schema | |
-| | * Lint Docker
compose files with JSON Schema | |
-| | * Lint
chart/values.schema.json file with JSON Schema | |
-| | * Lint
chart/values.yaml file with JSON Schema | |
-| | * Lint
airflow/config_templates/config.yml file with JSON Schema | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| lint-markdown | Run markdownlint
| |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| lint-openapi | * Lint OpenAPI
using spectral | |
-| | * Lint OpenAPI
using openapi-spec-validator | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| mixed-line-ending | Detect if mixed
line ending is used (\r vs. \r\n) | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| mypy-core | Run mypy for
core | * |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| mypy-dev | Run mypy for dev
| * |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| mypy-docs | Run mypy for
/docs/ folder | * |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| mypy-providers | Run mypy for
providers | * |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| pretty-format-json | Format JSON
files | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| python-no-log-warn | Check if there
are no deprecate log warn | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| replace-bad-characters | Replace bad
characters | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| rst-backticks | Check if RST
files use double backticks for code | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| ruff | ruff
| |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| shellcheck | Check Shell
scripts syntax correctness | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| trailing-whitespace | Remove trailing
whitespace at end of line | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| ts-compile-format-lint-www | TS types
generation and ESLint/Prettier against current UI files | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| update-black-version | Update black
versions everywhere | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| update-breeze-cmd-output | Update output of
breeze commands in BREEZE.rst | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| update-breeze-readme-config-hash | Update Breeze
README.md with config files hash | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| update-common-sql-api-stubs | Check and update
common.sql API stubs | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| update-er-diagram | Update ER
diagram | * |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| update-extras | Update extras in
documentation | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| update-in-the-wild-to-be-sorted | Sort
INTHEWILD.md alphabetically | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| update-inlined-dockerfile-scripts | Inline
Dockerfile and Dockerfile.ci scripts | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| update-installed-providers-to-be-sorted | Sort
alphabetically and uniquify installed_providers.txt | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| update-local-yml-file | Update mounts in
the local yml file | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| update-migration-references | Update migration
ref doc | * |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| update-providers-dependencies | Update
cross-dependencies for providers packages | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| update-spelling-wordlist-to-be-sorted | Sort
alphabetically and uniquify spelling_wordlist.txt | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| update-supported-versions | Updates
supported versions in documentation | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| update-vendored-in-k8s-json-schema | Vendor k8s
definitions into values.schema.json | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| update-version | Update version
to the latest version in the documentation | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
-| yamllint | Check YAML files
with yamllint | |
-+-----------------------------------------------------------+------------------------------------------------------------------+---------+
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| ID | Description
| Image |
++===========================================================+==============================================================+=========+
+| black | Run black
(Python formatter) | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| blacken-docs | Run black on
Python code blocks in documentation files | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-aiobotocore-optional | Check if
aiobotocore is an optional dependency only | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-airflow-config-yaml-consistent | Check
consistency between config.yml and default_config.cfg | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-airflow-provider-compatibility | Check
compatibility of Providers with Airflow | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-apache-license-rat | Check if
licenses are OK for Apache | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-base-operator-partial-arguments | Check
BaseOperator and partial() arguments | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-base-operator-usage | * Check
BaseOperator[Link] core imports | |
+| | * Check
BaseOperator[Link] other imports | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-boring-cyborg-configuration | Checks for
Boring Cyborg configuration consistency | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-breeze-top-dependencies-limited | Breeze should
have small number of top-level dependencies | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-builtin-literals | Require literal
syntax when initializing builtin types | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-changelog-has-no-duplicates | Check changelogs
for duplicate entries | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-core-deprecation-classes | Verify usage of
Airflow deprecation classes in core | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-daysago-import-from-utils | Make sure
days_ago is imported from airflow.utils.dates | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-decorated-operator-implements-custom-name | Check @task
decorator implements custom_operator_name | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-docstring-param-types | Check that
docstrings do not specify param types | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-example-dags-urls | Check that
example dags url include provider versions | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-executables-have-shebangs | Check that
executables have shebang | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-extra-packages-references | Checks setup
extra packages | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-extras-order | Check order of
extras in Dockerfile | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-for-inclusive-language | Check for
language that we do not accept as community | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-hooks-apply | Check if all
hooks apply to the repository | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-incorrect-use-of-LoggingMixin | Make sure
LoggingMixin is not used alone | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-init-decorator-arguments | Check model
__init__ and decorator arguments are in sync | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-lazy-logging | Check that all
logging methods are lazy | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-links-to-example-dags-do-not-use-hardcoded-versions | Verify example
dags do not use hard-coded version numbers | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-merge-conflict | Check that merge
conflicts are not being committed | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-newsfragments-are-valid | Check
newsfragments are valid | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-no-airflow-deprecation-in-providers | Do not use
DeprecationWarning in providers | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-no-providers-in-core-examples | No providers
imports in core example DAGs | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-no-relative-imports | No relative
imports | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-only-new-session-with-provide-session | Check
NEW_SESSION is only used with @provide_session | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-persist-credentials-disabled-in-github-workflows | Check that
workflow files have persist-credentials disabled | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-pre-commit-information-consistent | Validate hook
IDs & names and sync with docs | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-provide-create-sessions-imports | Check
provide_session and create_session imports | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-provider-yaml-valid | Validate
provider.yaml files | * |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-providers-init-file-missing | Provider init
file is missing | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-providers-subpackages-init-file-exist | Provider
subpackage init files are there | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-pydevd-left-in-code | Check for pydevd
debug statements accidentally left | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-revision-heads-map | Check that the
REVISION_HEADS_MAP is up-to-date | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-safe-filter-usage-in-html | Don't use safe
in templates | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-setup-order | Check order of
dependencies in setup.cfg and setup.py | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-start-date-not-used-in-defaults | start_date not
to be defined in default_args in example_dags | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-system-tests-present | Check if system
tests have required segments of code | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-system-tests-tocs | Check that
system tests is properly added | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-tests-unittest-testcase | Check that unit
tests do not inherit from unittest.TestCase | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-urlparse-usage-in-code | Don't use
urlparse in code | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| check-xml | Check XML files
with xmllint | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| codespell | Run codespell to
check for common misspellings in files | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| compile-www-assets | Compile www
assets | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| compile-www-assets-dev | Compile www
assets in dev mode | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| create-missing-init-py-files-tests | Create missing
init.py files in tests | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| debug-statements | Detect
accidentally committed debug statements | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| detect-private-key | Detect if
private key is added to the repository | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| doctoc | Add TOC for
Markdown and RST files | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| end-of-file-fixer | Make sure that
there is an empty line at the end | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| fix-encoding-pragma | Remove encoding
header from Python files | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| flynt | Run flynt string
format converter for Python | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| identity | Print input to
the static check hooks for troubleshooting | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| insert-license | * Add license
for all SQL files | |
+| | * Add license
for all RST files | |
+| | * Add license
for all CSS/JS/JSX/PUML/TS/TSX files | |
+| | * Add license
for all JINJA template files | |
+| | * Add license
for all Shell files | |
+| | * Add license
for all Python files | |
+| | * Add license
for all XML files | |
+| | * Add license
for all Helm template files | |
+| | * Add license
for all YAML files except Helm templates | |
+| | * Add license
for all Markdown files | |
+| | * Add license
for all other files | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| lint-chart-schema | Lint
chart/values.schema.json file | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| lint-css | stylelint
| |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| lint-dockerfile | Lint Dockerfile
| |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| lint-helm-chart | Lint Helm Chart
| |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| lint-json-schema | * Lint JSON
Schema files with JSON Schema | |
+| | * Lint NodePort
Service with JSON Schema | |
+| | * Lint Docker
compose files with JSON Schema | |
+| | * Lint
chart/values.schema.json file with JSON Schema | |
+| | * Lint
chart/values.yaml file with JSON Schema | |
+| | * Lint
config_templates/config.yml file with JSON Schema | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| lint-markdown | Run markdownlint
| |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| lint-openapi | * Lint OpenAPI
using spectral | |
+| | * Lint OpenAPI
using openapi-spec-validator | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| mixed-line-ending | Detect if mixed
line ending is used (\r vs. \r\n) | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| mypy-core | Run mypy for
core | * |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| mypy-dev | Run mypy for dev
| * |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| mypy-docs | Run mypy for
/docs/ folder | * |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| mypy-providers | Run mypy for
providers | * |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| pretty-format-json | Format JSON
files | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| python-no-log-warn | Check if there
are no deprecate log warn | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| replace-bad-characters | Replace bad
characters | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| rst-backticks | Check if RST
files use double backticks for code | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| ruff | ruff
| |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| shellcheck | Check Shell
scripts syntax correctness | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| trailing-whitespace | Remove trailing
whitespace at end of line | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| ts-compile-format-lint-www | TS types
generation / ESLint / Prettier against UI files | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| update-black-version | Update black
versions everywhere | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| update-breeze-cmd-output | Update output of
breeze commands in BREEZE.rst | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| update-breeze-readme-config-hash | Update Breeze
README.md with config files hash | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| update-common-sql-api-stubs | Check and update
common.sql API stubs | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| update-er-diagram | Update ER
diagram | * |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| update-extras | Update extras in
documentation | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| update-in-the-wild-to-be-sorted | Sort
INTHEWILD.md alphabetically | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| update-inlined-dockerfile-scripts | Inline
Dockerfile and Dockerfile.ci scripts | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| update-installed-providers-to-be-sorted | Sort
alphabetically and uniquify installed_providers.txt | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| update-local-yml-file | Update mounts in
the local yml file | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| update-migration-references | Update migration
ref doc | * |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| update-providers-dependencies | Update
cross-dependencies for providers packages | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| update-spelling-wordlist-to-be-sorted | Sort
alphabetically and uniquify spelling_wordlist.txt | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| update-supported-versions | Updates
supported versions in documentation | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| update-vendored-in-k8s-json-schema | Vendor k8s
definitions into values.schema.json | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| update-version | Update version
to the latest version in the documentation | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
+| yamllint | Check YAML files
with yamllint | |
++-----------------------------------------------------------+--------------------------------------------------------------+---------+
.. END AUTO-GENERATED STATIC CHECK LIST
