TrueCloudLab/frostfs-node
20
1
Fork 27
Code Issues 94 Pull requests 4 Projects Releases 1 Activity Actions

fix gitlint run in CI #158

Merged
realloc merged 1 commit from realloc/frostfs-node:fix/gitlint into master 2023-03-22 08:58:35 +00:00
Conversation 0 Commits 1 Files changed 3 +4 -3
3 changed files with 4 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
.gitlint
View file

@ -1,6 +1,6 @@
[general] [general]
fail-without-commits=true fail-without-commits=True
fyrchik commented 2023-03-22 08:31:02 +00:00
Review
Copy link

The official doc mentions small letters, do we need this change?
https://jorisroovers.com/gitlint/configuration/

The official doc mentions small letters, do we need this change? https://jorisroovers.com/gitlint/configuration/
realloc commented 2023-03-22 08:36:59 +00:00
Review
Copy link

Agree, but the internal representation uses capital letters, so the warning shown from binary request changes with capital letters =) Both ways work.

DEBUG: gitlint.cli Configuration
config-path: /home/realloc/projects/tcl-realloc/frostfs-node/.gitlint
[GENERAL]
extra-path: None
contrib: ['CC1']
ignore: 
ignore-merge-commits: True
ignore-fixup-commits: True
ignore-fixup-amend-commits: True
ignore-squash-commits: True
ignore-revert-commits: True
ignore-stdin: False
staged: False
fail-without-commits: True
regex-style-search: True
verbosity: 3
debug: True
Agree, but the internal representation uses capital letters, so the warning shown from binary request changes with capital letters =) Both ways work. ``` DEBUG: gitlint.cli Configuration config-path: /home/realloc/projects/tcl-realloc/frostfs-node/.gitlint [GENERAL] extra-path: None contrib: ['CC1'] ignore: ignore-merge-commits: True ignore-fixup-commits: True ignore-fixup-amend-commits: True ignore-squash-commits: True ignore-revert-commits: True ignore-stdin: False staged: False fail-without-commits: True regex-style-search: True verbosity: 3 debug: True ```
regex-style-search=true regex-style-search=True
contrib=CC1 contrib=CC1
[title-match-regex] [title-match-regex]

1
.pre-commit-config.yaml
View file

@ -7,6 +7,7 @@ repos:
hooks: hooks:
- id: gitlint - id: gitlint
stages: [commit-msg] stages: [commit-msg]
- id: gitlint-ci
- repo: https://github.com/pre-commit/pre-commit-hooks - repo: https://github.com/pre-commit/pre-commit-hooks
rev: v4.4.0 rev: v4.4.0

2
.woodpecker/pre-commit.yml
View file

@ -8,7 +8,7 @@ pipeline:
image: git.frostfs.info/truecloudlab/frostfs-ci:v0.36 image: git.frostfs.info/truecloudlab/frostfs-ci:v0.36
commands: commands:
- export HOME="$(getent passwd $(id -u) | cut '-d:' -f6)" - export HOME="$(getent passwd $(id -u) | cut '-d:' -f6)"
- pre-commit run - pre-commit run --hook-stage manual
unit: unit:
image: git.frostfs.info/truecloudlab/frostfs-ci:v0.36 image: git.frostfs.info/truecloudlab/frostfs-ci:v0.36