This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/master by this push:
new 99c2e03 Add identity pre-commit hook (#13089)
99c2e03 is described below
commit 99c2e03da9b552b74d78b1b10ce9b8f85c686678
Author: Ruben Laguna <[email protected]>
AuthorDate: Wed Dec 16 09:03:48 2020 +0100
Add identity pre-commit hook (#13089)
---
.pre-commit-config.yaml | 1 +
BREEZE.rst | 26 +++++++++++++++++---------
STATIC_CODE_CHECKS.rst | 2 ++
breeze-complete | 1 +
4 files changed, 21 insertions(+), 9 deletions(-)
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index c423fb2..12a05b4 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -148,6 +148,7 @@ repos:
- "2"
- repo: meta
hooks:
+ - id: identity
- id: check-hooks-apply
- repo: https://github.com/psf/black
rev: 20.8b1
diff --git a/BREEZE.rst b/BREEZE.rst
index 238b6c0..b42d3d7 100644
--- a/BREEZE.rst
+++ b/BREEZE.rst
@@ -751,6 +751,14 @@ The above will run mypy check for all files.
</a>
</div>
+If you want ever need to get a list of the files that will be checked (for
troubleshooting when playing with the
+``--from-ref`` and ``--to-ref``
+
+.. code-block:: bash
+
+ breeze static-check identity --verbose # currently staged files
+ breeze static-check identity --verbose -- --from-ref $(git merge-base
master HEAD) --to-ref HEAD # branch updates
+
Building the Documentation
--------------------------
@@ -2144,15 +2152,15 @@ This is the current syntax for `./breeze <./breeze>`_:
check-executables-have-shebangs check-hooks-apply
check-integrations
check-merge-conflict check-xml consistent-pylint
daysago-import-check
debug-statements detect-private-key doctoc
dont-use-safe-filter end-of-file-fixer
- fix-encoding-pragma flake8 forbid-tabs helm-lint
incorrect-use-of-LoggingMixin
- insert-license isort json-schema language-matters
lint-dockerfile lint-openapi
- markdownlint mermaid mixed-line-ending mypy mypy-helm
no-providers-in-core-examples
- no-relative-imports pre-commit-descriptions
provide-create-sessions
- providers-init-file provider-yamls pydevd pydocstyle pylint
pylint-tests
- python-no-log-warn pyupgrade restrict-start_date
rst-backticks setup-order
- setup-extra-packages shellcheck sort-in-the-wild stylelint
trailing-whitespace
- update-breeze-file update-extras update-local-yml-file
update-setup-cfg-file
- version-sync yamllint
+ fix-encoding-pragma flake8 forbid-tabs helm-lint identity
+ incorrect-use-of-LoggingMixin insert-license isort
json-schema language-matters
+ lint-dockerfile lint-openapi markdownlint mermaid
mixed-line-ending mypy mypy-helm
+ no-providers-in-core-examples no-relative-imports
pre-commit-descriptions
+ provide-create-sessions providers-init-file provider-yamls
pydevd pydocstyle pylint
+ pylint-tests python-no-log-warn pyupgrade restrict-start_date
rst-backticks
+ setup-order setup-extra-packages shellcheck sort-in-the-wild
stylelint
+ trailing-whitespace update-breeze-file update-extras
update-local-yml-file
+ update-setup-cfg-file version-sync yamllint
You can pass extra arguments including options to to the pre-commit
framework as
<EXTRA_ARGS> passed after --. For example:
diff --git a/STATIC_CODE_CHECKS.rst b/STATIC_CODE_CHECKS.rst
index 6c79d41..e8a6867 100644
--- a/STATIC_CODE_CHECKS.rst
+++ b/STATIC_CODE_CHECKS.rst
@@ -106,6 +106,8 @@ require Breeze Docker images to be installed locally:
-----------------------------------
---------------------------------------------------------------- ------------
``helm-lint`` Verifies if helm lint passes for the
chart
-----------------------------------
---------------------------------------------------------------- ------------
+``identity`` Prints inputs to the static check hooks
for troubleshooting
+-----------------------------------
---------------------------------------------------------------- ------------
``incorrect-use-of-LoggingMixin`` Checks if LoggingMixin is properly
imported.
-----------------------------------
---------------------------------------------------------------- ------------
``insert-license`` Adds licenses for most file types.
diff --git a/breeze-complete b/breeze-complete
index 6bbb51a..9bff03c 100644
--- a/breeze-complete
+++ b/breeze-complete
@@ -92,6 +92,7 @@ fix-encoding-pragma
flake8
forbid-tabs
helm-lint
+identity
incorrect-use-of-LoggingMixin
insert-license
isort
