| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
| |
Change-Id: I832ca2f115ed1d2b1471d7b00ddf36cdeda41ad1
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The goal here is to avoid conflicts between flake8 and hacking version each
2 days.
Inspired from nova's approach [1].
The flake8 version to install will be determined by hacking and
requirements will stay aligned instead of relying on different versions.
[1] https://opendev.org/openstack/nova/src/branch/master/.pre-commit-config.yaml#L26-L35
[2] https://opendev.org/openstack/hacking/src/branch/master/requirements.txt#L1
Change-Id: Iae20f13ba6293b94b5bd131941263e097eba0518
|
|
|
|
|
|
|
|
|
|
|
| |
The conflict is caused by:
The user requested flake8 3.8.3 (from /home/zuul/.cache/pre-commit/repozi7hytk7)
hacking 3.0.1 depends on flake8<3.8.0 and >=3.6.0
Install the latest version of hacking to fix the conflict error with the
pre-commit hook.
Change-Id: If1c54502e6e10751fd722694aa5429a08f376798
|
|
Introduced changes:
- pre-commit config and rules
- Add pre-commit to pep8 gate, Flake8 is covered in the pre-commit
hooks.
- Applying fixes for pre-commit compliance in all code.
Also commit hash will be used instead of version tags in pre-commit to
prevend arbitrary code from running in developer's machines.
pre-commit will be used to:
- trailing whitespace;
- Replaces or checks mixed line ending (mixed-line-ending);
- Forbid files which have a UTF-8 byte-order marker
(check-byte-order-marker);
- Checks that non-binary executables have a proper
shebang (check-executables-have-shebangs);
- Check for files that contain merge conflict strings
(check-merge-conflict);
- Check for debugger imports and py37+ breakpoint()
calls in python source (debug-statements);
- Attempts to load all yaml files to verify syntax (check-yaml);
- Run flake8 checks (flake8) (local)
For further details about tests please refer to:
https://github.com/pre-commit/pre-commit-hooks
Change-Id: Id03c1ea4752db3236c36521ca9c67125d5ba2526
Co-authored-by: Moisés Guimarães de Medeiros <moguimar@redhat.com>
|