conventional-changelog/commitlint

Camel-case is confused with sentence-case / start-case

Open

#3501 aperta il 13 gen 2023

Vedi su GitHub
 (0 commenti) (1 reazione) (0 assegnatari)TypeScript (896 fork)batch import
bughelp wanted

Metriche repository

Star
 (15.497 star)
Metriche merge PR
 (Merge medio 5g 19h) (65 PR mergiate in 30 g)

Descrizione

Expected Behavior

If only sentence-case and start-case are denied from scopes, I'd expect camel-case to work.

Current Behavior

With

  scope-case:
    - 2
    - never
    - - sentence-case
      - start-case

I get

⧗   input: fix(ClearlyDefined): Make `Described` use `CurationFacets`

These are glob lists, not facet objects.

Signed-off-by: Sebastian Schuberth <sschuberth@gmail.com>
✖   scope must not be sentence-case, start-case [scope-case]

Affected packages

  • cli
  • core
  • prompt
  • config-angular

Possible Solution

When checking for sentence-case / start-case, ensure that there is more than a single word.

Steps to Reproduce

See https://github.com/oss-review-toolkit/ort/actions/runs/3909253534/jobs/6680143679#step:4:17

Context

No response

commitlint --version

17.4.2

git --version

2.39.0

node --version

16.14.2

Guida contributor

Direzione di ricerca
Ispeziona l'implementazione della regola scope case in commitlint. Trova dove vengono controllati sentence case e start case. Assicurati che il rilevamento richieda più di una parola per evitare falsi positivi con camel case.
Tech stack
typescript
Dominio
clitooling
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
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.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitNode.jsTypeScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65