Skip to content

fix: I fallimenti dell'hook pre-commit ora interrompono l'esecuzione#18

Open
moris88 wants to merge 1 commit into
mainfrom
refactor/pre-commit-hook-output
Open

fix: I fallimenti dell'hook pre-commit ora interrompono l'esecuzione#18
moris88 wants to merge 1 commit into
mainfrom
refactor/pre-commit-hook-output

Conversation

@moris88

@moris88 moris88 commented Aug 27, 2025

Copy link
Copy Markdown
Owner

Questo commit si concentra principalmente sul miglioramento della robustezza dell'esecuzione dell'hook di pre-commit e sull'aumento della leggibilità del file README.md.

Le modifiche principali includono:

  • Gestione degli errori dell'hook di pre-commit migliorata:

    • In precedenza, se un comando pre-commit falliva, il flusso di lavoro registrava un avviso e continuava. Questo comportamento è stato modificato per interrompere l'esecuzione immediatamente al fallimento di un comando pre-commit.
    • Quando un comando pre-commit fallisce, l'output da stdout e stderr del comando fallito viene ora catturato e stampato, fornendo informazioni cruciali per il debug all'utente. Ciò assicura che gli sviluppatori siano immediatamente consapevoli dei problemi con i loro hook pre-commit e possano risolverli prima di procedere con un commit.
    • Il messaggio di avviso corrispondente in locales/en.json e locales/it.json è stato aggiornato per riflettere questa modifica, indicando chiaramente che il processo si fermerà in caso di fallimento.
  • Miglioramenti alla leggibilità di README.md:

    • Sono state rimosse linee vuote superflue all'interno delle sezioni per rendere il contenuto meno distanziato e più coeso.
    • È stato standardizzato lo stile dei punti elenco da * Elemento a * Elemento per coerenza tra tutti gli elenchi nelle sezioni inglese e italiana. Ciò migliora l'uniformità visiva e l'aderenza alla comune formattazione Markdown.

Questo commit si concentra principalmente sul miglioramento della robustezza dell'esecuzione dell'hook di pre-commit e sull'aumento della leggibilità del file `README.md`.

Le modifiche principali includono:

*   **Gestione degli errori dell'hook di pre-commit migliorata**:
    *   In precedenza, se un comando pre-commit falliva, il flusso di lavoro registrava un avviso e continuava. Questo comportamento è stato modificato per interrompere l'esecuzione immediatamente al fallimento di un comando pre-commit.
    *   Quando un comando pre-commit fallisce, l'output da `stdout` e `stderr` del comando fallito viene ora catturato e stampato, fornendo informazioni cruciali per il debug all'utente. Ciò assicura che gli sviluppatori siano immediatamente consapevoli dei problemi con i loro hook pre-commit e possano risolverli prima di procedere con un commit.
    *   Il messaggio di avviso corrispondente in `locales/en.json` e `locales/it.json` è stato aggiornato per riflettere questa modifica, indicando chiaramente che il processo si fermerà in caso di fallimento.

*   **Miglioramenti alla leggibilità di README.md**:
    *   Sono state rimosse linee vuote superflue all'interno delle sezioni per rendere il contenuto meno distanziato e più coeso.
    *   È stato standardizzato lo stile dei punti elenco da `*   Elemento` a `* Elemento` per coerenza tra tutti gli elenchi nelle sezioni inglese e italiana. Ciò migliora l'uniformità visiva e l'aderenza alla comune formattazione Markdown.
@moris88 moris88 self-assigned this Aug 27, 2025
@github-actions

Copy link
Copy Markdown

📦 Modifiche nel diff con main: 📝 4 file cambiati, ➕ 79 righe aggiunte, ➖ 79 rimosse ⚖️

⚠️ Duplicazione codice rilevata: 0.68% (📉 -6.55%) in 1 blocco duplicato

📊 Statistiche sintetiche (per formato)
Formato Files Linee Cloni Righe duplicate (%)
javascript 1 420 1 6 (1.43%)
json 2 202 0 0 (0.00%)
markdown 1 258 0 0 (0.00%)
Total 4 880 1 6 (0.68%)
📄 Elenco duplicati (espandi per vedere i dettagli)
  • src/main.jssrc/main.js (Linee 259-265 ↔ 238-244)
() {
  await initialize()
  const branches = getLocalBranches()
    .split('\n')
    .map((b) => b.trim())
    .filter((b) => b && !b.startsWith('*'))
  const targetBranch = await selectBranchToCheckout

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant