In nodejs/node#42934, I was trying to revert nodejs/node#27371, so git revert automatically created this commit title for me:
Revert "bootstrap: delay the instantiation of maps in per-context scripts"
which has 74 characters. Since it exceeds our maximum limit of 72 characters, the commit message linter failed on CI - https://github.com/nodejs/node/runs/6248053065?check_suite_focus=true#step:6:15
Details
Run echo "::add-matcher::.github/workflows/commit-lint-problem-matcher.json"
npm WARN exec The following package was not found and will be installed: core-validate-commit
# b04fe688d5859f707cf1a5e0206967268118bf7a
ok 1 co-authored-by-is-trailer: no Co-authored-by metadata
ok 2 fixes-url: skipping fixes-url # SKIP
ok 3 line-after-title: blank line after title
ok 4 line-length: line-lengths are valid
ok 5 subsystem: valid subsystems [bootstrap]
ok 6 title-format: Title is formatted correctly.
Error: not ok 7 title-length: Title must be <= 72 columns.
---
{
found: 74,
compare: '<=',
wanted: 72,
at: {
line: 0,
column: 72,
body: 'Revert "bootstrap: delay the instantiation of maps in per-context scripts"'
}
}
...
0..7
# tests 7
# pass 5
# fail 1
# Please review the commit message guidelines:
# https://github.com/nodejs/node/blob/HEAD/doc/contributing/pull-requests.md#commit-message-guidelines
As a workaround, I had to change the commit title to
bootstrap: stop delaying instantiation of maps in per-context scripts
which has 69 characters (< 72) but it isn't satisfying anymore. :/
Instead of failing, IMO the linter should accept standard revert commit titles regardless of the size.
In nodejs/node#42934, I was trying to revert nodejs/node#27371, so git revert automatically created this commit title for me:
which has 74 characters. Since it exceeds our maximum limit of 72 characters, the commit message linter failed on CI - https://github.com/nodejs/node/runs/6248053065?check_suite_focus=true#step:6:15
DetailsRun echo "::add-matcher::.github/workflows/commit-lint-problem-matcher.json" npm WARN exec The following package was not found and will be installed: core-validate-commit # b04fe688d5859f707cf1a5e0206967268118bf7a ok 1 co-authored-by-is-trailer: no Co-authored-by metadata ok 2 fixes-url: skipping fixes-url # SKIP ok 3 line-after-title: blank line after title ok 4 line-length: line-lengths are valid ok 5 subsystem: valid subsystems [bootstrap] ok 6 title-format: Title is formatted correctly. Error: not ok 7 title-length: Title must be <= 72 columns. --- { found: 74, compare: '<=', wanted: 72, at: { line: 0, column: 72, body: 'Revert "bootstrap: delay the instantiation of maps in per-context scripts"' } } ... 0..7 # tests 7 # pass 5 # fail 1 # Please review the commit message guidelines: # https://github.com/nodejs/node/blob/HEAD/doc/contributing/pull-requests.md#commit-message-guidelinesAs a workaround, I had to change the commit title to
which has 69 characters (< 72) but it isn't satisfying anymore. :/
Instead of failing, IMO the linter should accept standard revert commit titles regardless of the size.