Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enforce formatting on helm chart json schema files #15895

Merged
merged 1 commit into from May 17, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
3 changes: 3 additions & 0 deletions .editorconfig
Expand Up @@ -47,3 +47,6 @@ indent_size = 2

[*.{htm,html}]
indent_size = 2

[*.json]
indent_size = 4
8 changes: 8 additions & 0 deletions .pre-commit-config.yaml
Expand Up @@ -180,6 +180,14 @@ repos:
exclude: ^airflow/_vendor/
args:
- --remove
- id: pretty-format-json
args:
- --autofix
- --no-sort-keys
- --indent
- "4"
files: ^chart/values\.schema\.json$|^chart/values_schema\.schema\.json$
pass_filenames: true
- repo: https://github.com/asottile/pyupgrade
rev: v2.13.0
hooks:
Expand Down
4 changes: 2 additions & 2 deletions BREEZE.rst
Expand Up @@ -2265,8 +2265,8 @@ This is the current syntax for `./breeze <./breeze>`_:
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
pre-commit-hook-names provide-create-sessions providers-init-file provider-yamls
pydevd pydocstyle pylint pylint-tests python-no-log-warn pyupgrade
pre-commit-hook-names pretty-format-json 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 sort-spelling-wordlist stylelint trailing-whitespace ui-lint
update-breeze-file update-extras update-local-yml-file update-setup-cfg-file
Expand Down
2 changes: 2 additions & 0 deletions STATIC_CODE_CHECKS.rst
Expand Up @@ -138,6 +138,8 @@ require Breeze Docker images to be installed locally:
----------------------------------- ---------------------------------------------------------------- ------------
``pre-commit-hook-names`` Check that hook names are not overly long
----------------------------------- ---------------------------------------------------------------- ------------
``pretty-format-json`` Formats json files
----------------------------------- ---------------------------------------------------------------- ------------
``provide-create-sessions`` Make sure provide-session and create-session imports are OK
----------------------------------- ---------------------------------------------------------------- ------------
``providers-init-file`` Check that provider's __init__.py file is removed
Expand Down
1 change: 1 addition & 0 deletions breeze-complete
Expand Up @@ -121,6 +121,7 @@ no-providers-in-core-examples
no-relative-imports
pre-commit-descriptions
pre-commit-hook-names
pretty-format-json
provide-create-sessions
providers-init-file
provider-yamls
Expand Down