fix: Body line with reference like `#123` is considered footer

Metriche repository

Star

15.497 stelle (15.497 stelle)

Metriche merge PR

Merge medio 12h 56m (Merge medio 12h 56m)52 PR mergiate in 30 g (52 PR mergiate in 30 g)

Descrizione

echo 'rules: {footer-leading-blank: [2, always]}' > .commitlintrc.yml
cat > commit-message.txt << EOF
fix: don't frobnicate the foobar

This commit prevents the foobar from being frobnicated
during operation, and therefore fixes #69.
EOF
npx '@commitlint/cli@19.3.0' -e commit-message.txt

⧗   input: fix: don't frobnicate the foobar

This commit prevents the foobar from being frobnicated

during operation, and therefore fixes #69.
✖   footer must have leading blank line [footer-leading-blank]

✖   found 1 problems, 0 warnings
ⓘ   Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint

Guida contributor

Direzione di ricerca

Investigate the commit message parsing logic in the @commitlint/parse package, specifically the function that separates body and footer. The current implementation treats any line starting with a reference (like #123 or user/repo#123) as footer, which is incorrect. Review the regex or conditional logic to ensure that reference lines within the body are not mistaken for footer lines. Check for existing tests or comments in the codebase related to footer detection.

Tech stack

typescriptnodejs

Dominio

cli

Tipo issue

Bug

DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.

3

Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.

1-3 ore

Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.

Attiva

ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.

Chiara

Prerequisiti

commit message conventionscommitlint configurationNode.jsTypeScript

Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.

65