fix(hooks): pre-push reminder misses pushes in subshells, bash -c and xargs
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 57/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- javascript, node.js
- Ambito
- developer-experience, tooling
Direzione di ricerca
Start by reading the pre-push hook and its existing tests to see how commands are matched. Add coverage for the listed wrappers, nested shell commands, and subshells, then update the README limits paragraph. Done means each listed form triggers the reminder and the tests pass.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
What
The pre-push hook finds git push at the start of a command, after &&, ; or cd, and with git -C. A push inside ( ... ), if ...; then git push; fi, bash -c "git push", env, time, command or xargs gets no reminder.
Why
The hook is opt-in and advisory, so this is low impact, but a missed push silently skips the review it promises.
Acceptance criteria
- Known wrappers (
env,time,command,sudo,nohup) are stripped before matching. -
bash -c/sh -cstrings and subshells are parsed one level deep. - Tests cover each form; the README limits paragraph is updated.
- Lingua principale
- JavaScript
- Stelle
- 0
- Fork
- 0
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di devmaxxx/stackwise
-
feat: add a plugin iconApertaenhancement good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
-
enhancement
Difficoltà 3/5 1-2 giorni Idoneità per principianti 55/100
-
enhancement
Difficoltà 3/5 1-2 giorni Idoneità per principianti 68/100
-
bug eval
Difficoltà 3/5 1-2 giorni Idoneità per principianti 66/100
-
bug
Difficoltà 3/5 1-2 giorni Idoneità per principianti 55/100
Tutte le issue di devmaxxx/stackwise
Issue simili
-
[bug] extension mappingAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 64/100
-
bug code-annotation
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
quarto-dev/quarto-cli#15215 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 70/100
521xueweihan/HelloGitHub#4133 ·
-
Difficoltà 1/5 1-3 ore Idoneità per principianti 88/100
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
Hugo-SA/receita-random#20 ·
I maintainer di solito rispondono entro 1 giorno