fix(hooks): pre-push reminder misses pushes in subshells, bash -c and xargs
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 4/5
- Tempo estimado
- 3-5 dias
- Facilidade para iniciantes
- 57/100
- Tipo de issue
- Bug
- Clareza
- Claramente especificada
- Status de atividade
- Ativa
- Stack de tecnologia
- javascript, node.js
- Domínio
- developer-experience, tooling
Direção de pesquisa
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.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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.
- Linguagem predominante
- JavaScript
- Estrelas
- 0
- Forks
- 0
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Preparar o ambiente
Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de devmaxxx/stackwise
-
feat: add a plugin iconAbertaenhancement good first issue
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
-
enhancement
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 55/100
-
enhancement
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 68/100
-
bug eval
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 66/100
-
bug
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 55/100
Todas as issues de devmaxxx/stackwise
Issues semelhantes
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
AOSSIE-Org/BabyNest#284 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 85/100
seek4science/seek#2762 ·
Mantenedores costumam responder em até 5 dias
-
feedback simulation workshop
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 64/100
githubnext/gh-aw-workshop#5009 ·
Mantenedores costumam responder em até 1 dia
-
Clinic sharing: fast sync refusal message suggests Delete Data, which only developer builds haveAbertabug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
avniproject/avni-client#2228 ·
-
Add: K+ Sport 2 LogoAbertacheck:passed logos:add
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 85/100
Mantenedores costumam responder em até 6 dias