-
-
Notifications
You must be signed in to change notification settings - Fork 52
Closed
Description
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
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.
Metadata
Metadata
Assignees
Labels
No labels