hound-search/hound

Add ability to turn of "git clone --depth=1", it's sometimes quite expensive

Open

#207 aberto em 10 de mai. de 2016

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)JavaScript (606 forks)batch import
enhancementhelp wanted

Métricas do repositório

Stars
 (5.470 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

This was changed in 2e9ceb9 by @kellegous. See this comment to a CocoaPods issue for why using shallow clones for things like this is a bad idea.

IMO this commit should just be reverted, it makes much more sense to do a full clone and then fetch that with the way the fetch protocol works and how shallow clones can trigger bad edgecases in it, but if not please add some config option to turn this off.

Edit: FWIW this isn't some theoretical thing, I have a very large repo that's running into this edge-case, a "pull" should take 2-3s, but sometimes takes up to 6 minutes from hound (a fresh clone is ~3m) because of the edge cases triggered by the shallow clone.

Guia do colaborador

Direção de pesquisa
Encontre o código responsável pelo git clone no repositório, especificamente o commit 2e9ceb9 que introduziu o clone superficial. Entenda como o comando clone é construído e adicione uma opção de configuração (por exemplo, em variável de ambiente ou arquivo de configuração) para desabilitar a flag depth=1. Garanta que, quando a opção estiver definida, um clone completo seja executado.
Pilha de tecnologia
javascript
Domain
backend
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60