commitlint icon indicating copy to clipboard operation
commitlint copied to clipboard

bug: treats body with URL as footer

Open abitrolly opened this issue 1 year ago • 2 comments

Steps to Reproduce

1. Add this commit message:

fix: ingest - do not double strip /doc/ prefix

It is already stripped here
https://gitlab.com/gitlab-community/modelops/applied-ml/code-suggestions/ai-assist/-/blob/22bb9cf2b996dc2ec347bad761abb12fd11bcf75/scripts/ingest/gitlab-docs/steps/parse.rb#L16
  1. Watch commitlint failure for footer

Current Behavior

â§—   input: fix: ingest - do not double strip /doc/ prefix
It is already stripped here
https://gitlab.com/gitlab-community/modelops/applied-ml/code-suggestions/ai-assist/-/blob/22bb9cf2b996dc2ec347bad761abb12fd11bcf75/scripts/ingest/gitlab-docs/steps/parse.rb#L16
✖   footer's lines must not be longer than 100 characters [footer-max-line-length]
âš    footer must have leading blank line [footer-leading-blank]
✖   found 1 problems, 1 warnings

Expected Behavior

â§—   --- input ---
fix: ingest - do not double strip /doc/ prefix

It is already stripped here
https://gitlab.com/gitlab-community/modelops/applied-ml/code-suggestions/ai-assist/-/blob/22bb9cf2b996dc2ec347bad761abb12fd11bcf75/scripts/ingest/gitlab-docs/steps/parse.rb#L16
✖   body's lines must not be longer than 100 characters [body-max-line-length]
✖   found 1 warnings

Affected packages

  • [X] cli
  • [ ] core
  • [ ] prompt
  • [ ] config-angular

Possible Solution

Add the example to tests that cover bugs to be fixed.

Fix parser to return these lines in body.

Context

https://gitlab.com/gitlab-community/modelops/applied-ml/code-suggestions/ai-assist/-/jobs/7372910347

commitlint --version

i don't know

git --version

i don't know

node --version

i don't know

abitrolly avatar Jul 20 '24 17:07 abitrolly

Does this duplicate or relate to #4099, #3695, #3129?

escapedcat avatar Jul 22 '24 07:07 escapedcat

@escapedcat no. This one is about URL. Other issues have different inputs.

abitrolly avatar Jul 23 '24 06:07 abitrolly