conventional-changelog/commitlint

Acronyms incorrectly detected as upper-case

Open

#3312 aperta il 15 ago 2022

Vedi su GitHub
 (18 commenti) (0 reazioni) (0 assegnatari)TypeScript (896 fork)batch import
communitydiscussionhelp wanted

Metriche repository

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

Descrizione

Faulty Behavior

fix: JTD types -> JSON schema types

✖   subject must not be sentence-case, start-case, pascal-case, upper-case [subject-case]

The above fails as the linter detects the subject as being upper-case, even though it's just an acronym.

Affected packages

  • cli
  • core
  • prompt
  • config-angular

Possible Solution

Check that all letters in the word are uppercase and assume it's an acronym, ignoring any title-case lints. Skip for 1 letter words.

Steps to Reproduce (for bugs)

Lint the above commit message with @commitlint/config-conventional.

Your Environment

Executable Version
commitlint --version 17.0.3
git --version 2.34.1
node --version 16.15.1
@commitlint/config-conventional 17.0.3

Guida contributor

Direzione di ricerca
Indaga l'implementazione della regola 'subject case' in @commitlint/core, in particolare come funziona il rilevamento delle maiuscole. Modifica la logica per saltare le parole in cui tutte le lettere sono maiuscole (acronimi), a meno che la parola non sia una singola lettera.
Tech stack
typescriptnodejs
Dominio
developer experiencetooling
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitNode.js
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65