rewatch watcher: the lock-file exit path is duplicated
Mantenedores costumam responder em até 1 dia
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 2/5
- Tempo estimado
- 1-3 horas
- Facilidade para iniciantes
- 65/100
- Tipo de issue
- Refatoração
- Clareza
- Claramente especificada
- Status de atividade
- Ativa
- Stack de tecnologia
- rust
- Domínio
- build-system
Direção de pesquisa
Comece pelas duas cópias do bloco de saída em rewatch/src/watcher.rs, perto das linhas 393 e 635, e compare as verificações do arquivo de lock linha por linha. Mova a lógica compartilhada para uma única função auxiliar e faça com que os dois pontos de chamada usem essa função, sem alterar o comportamento. Está pronto quando os dois caminhos chamam a função auxiliar e os testes Rust existentes do rewatch (cargo test em rewatch/) continuam passando.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
Found by differential semantic drift analysis from ece8b148 to a7721303.
What is broken
In rewatch/src/watcher.rs, the block that removes the lock file and exits with "Exiting... (lockfile removed)" appears twice: at L393 and, added by #8730, at L635, each with its own check for the lock file. A future change to the exit behaviour has to be made in both places.
Possible cause (speculation only)
A guess: #8730 fixed the full-rebuild path by copying the existing exit block into it, rather than factoring it into a shared function.
- Linguagem predominante
- OCaml
- Estrelas
- 7.5k
- Forks
- 484
- Merge médio
- 19h 20min
- PRs com merge (30d)
- 96
Preparar o ambiente
Inicia o contêiner de desenvolvimento do projeto no navegador, com a sua própria conta do GitHub.
- Sem Dockerfile nem arquivo Docker Compose
- Sem modelo de pull request
- Ler o guia de contribuição
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 rescript-lang/rescript
-
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 35/100
rescript-lang/rescript#8806 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 35/100
rescript-lang/rescript#8798 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 3/5 Meio dia Facilidade para iniciantes 58/100
rescript-lang/rescript#8757 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 40/100
rescript-lang/rescript#8753 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 22/100
rescript-lang/rescript#8733 ·
Mantenedores costumam responder em até 1 dia
Todas as issues de rescript-lang/rescript
Issues semelhantes
-
[Bug] Dev image build fails: `npm install` in the root Dockerfile runs out of memory (exit 134)Talvez já em andamento @andest01 assumiu hoje. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
Mantenedores costumam responder em até 1 dia
-
100 points complexity:trivial drips-wave enhancement good first issue trivial
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
sorotrail/SoroBeacon#438 ·
Mantenedores costumam responder em até 1 dia
-
area: backend bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
snapotter-hq/SnapOtter#2415 ·
Mantenedores costumam responder em até 1 dia
-
100 points complexity:trivial drips-wave good first issue test trivial
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
Mantenedores costumam responder em até 1 dia
-
enhancement good first issue help wanted
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 85/100
Mantenedores costumam responder em até 1 dia