openstack/python-openstackclient
Code Issues Proposed changes
Files
7.1.2
python-openstackclient/.pre-commit-config.yaml

Ignoring revisions in .git-blame-ignore-revs. Click here to bypass and see the normal blame view.

41 lines
1.1 KiB
YAML
Raw Permalink Normal View History

Add pre-commit This is helpful to automate code style checks at runtime. We include documentation on how to run this as well as a general overview of style guidelines in OSC. Change-Id: I2dc5a0f760ce53269ae25677560b2611cc6bfd91 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
2021-03-11 16:20:15 +00:00
---
default_language_version:
# force all unspecified python hooks to run python3
python: python3
repos:
- repo: https://github.com/pre-commit/pre-commit-hooks
pre-commit: Bump versions We fold in the new black changes also. Change-Id: I326a0529b6b9f2aa9fbc33862567131839460797 Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
2024-04-23 12:22:26 +01:00
rev: v4.6.0
Add pre-commit This is helpful to automate code style checks at runtime. We include documentation on how to run this as well as a general overview of style guidelines in OSC. Change-Id: I2dc5a0f760ce53269ae25677560b2611cc6bfd91 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
2021-03-11 16:20:15 +00:00
hooks:
- id: trailing-whitespace
- id: mixed-line-ending
args: ['--fix', 'lf']
exclude: '.*\.(svg)$'
- id: check-byte-order-marker
- id: check-executables-have-shebangs
- id: check-merge-conflict
- id: debug-statements
- id: check-yaml
files: .*\.(yaml|yml)$
pre-commit: Allow unsafe YAML We're using tags in '.zuul.yaml', which requires use of the unsafe parser [1]. [1] https://stackoverflow.com/a/45967047 Change-Id: Ib97480f6fcaba7406ffe284e1935744346684b8b Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
2022-07-04 12:14:56 +01:00
args: ['--unsafe']
pre-commit: Add pyupgrade hook Another day, another useful hook. We also ignore the preceding patch that actually did the work, renaming the incorrect named file in the process. Change-Id: I412827761fbdeb36702ebaf5c1b727c62e629299 Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
2024-04-23 12:23:13 +01:00
- repo: https://github.com/asottile/pyupgrade
rev: v3.15.2
hooks:
- id: pyupgrade
args: ['--py38-plus']
pre-commit: Enable black Since our pep8 tox env now runs pre-commit, we can ensure that black styling will be maintained. Change-Id: Ifad1d67a83c4979b4f4de71de0124613d1133560 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
2023-05-08 11:41:29 +01:00
- repo: https://github.com/psf/black
pre-commit: Bump versions We fold in the new black changes also. Change-Id: I326a0529b6b9f2aa9fbc33862567131839460797 Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
2024-04-23 12:22:26 +01:00
rev: 24.4.0
pre-commit: Enable black Since our pep8 tox env now runs pre-commit, we can ensure that black styling will be maintained. Change-Id: Ifad1d67a83c4979b4f4de71de0124613d1133560 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
2023-05-08 11:41:29 +01:00
hooks:
- id: black
args: ['-S', '-l', '79']
Use pre-commit for 'pep8' tox target, bump versions We want to bump the versions of hacking, but doing so requires changes in two places: '.pre-commit-config.yaml' and 'tox.ini'. This is silly: we can simply use tox to handle pre-commit and leave all other dependencies to pre-commit. Do this, migrating bandit to pre-commit and bumping the other dependencies in the process. We remove the 'fast8' target and tool, since pre-commit is plenty fast here. Change-Id: Ifdc582ecc3b4479ea9ed9abcb16577c6e8b2f5c1 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
2023-05-08 11:12:16 +01:00
- repo: https://github.com/PyCQA/bandit
pre-commit: Bump versions We fold in the new black changes also. Change-Id: I326a0529b6b9f2aa9fbc33862567131839460797 Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
2024-04-23 12:22:26 +01:00
rev: 1.7.8
Use pre-commit for 'pep8' tox target, bump versions We want to bump the versions of hacking, but doing so requires changes in two places: '.pre-commit-config.yaml' and 'tox.ini'. This is silly: we can simply use tox to handle pre-commit and leave all other dependencies to pre-commit. Do this, migrating bandit to pre-commit and bumping the other dependencies in the process. We remove the 'fast8' target and tool, since pre-commit is plenty fast here. Change-Id: Ifdc582ecc3b4479ea9ed9abcb16577c6e8b2f5c1 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
2023-05-08 11:12:16 +01:00
hooks:
- id: bandit
tox: Remove bandit skips, run via pre-commit Most of these skips were unnecessary. The few that did generate warnings could be skipped. We also set 'skip_install' since there's no reason to build the package for linting purposes. Change-Id: I9644e5c19720b9c41c60e0a5882b7cd7f6a71f7b Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
2024-04-26 12:51:18 +01:00
args: ['-x', 'tests']
pre-commit: Bump linter versions We also migrate to the native hacking pre-commit hook. Change-Id: Idb5a825c20cb7f189997a4d6c3a9a88218fdb335 Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
2024-01-23 11:47:52 +00:00
- repo: https://opendev.org/openstack/hacking
rev: 6.1.0
Add pre-commit This is helpful to automate code style checks at runtime. We include documentation on how to run this as well as a general overview of style guidelines in OSC. Change-Id: I2dc5a0f760ce53269ae25677560b2611cc6bfd91 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
2021-03-11 16:20:15 +00:00
hooks:
pre-commit: Bump linter versions We also migrate to the native hacking pre-commit hook. Change-Id: Idb5a825c20cb7f189997a4d6c3a9a88218fdb335 Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
2024-01-23 11:47:52 +00:00
- id: hacking
additional_dependencies: []
Add pre-commit This is helpful to automate code style checks at runtime. We include documentation on how to run this as well as a general overview of style guidelines in OSC. Change-Id: I2dc5a0f760ce53269ae25677560b2611cc6bfd91 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
2021-03-11 16:20:15 +00:00
exclude: '^(doc|releasenotes|tools)/.*$'
Copy Permalink