miickel/gulp-angular-templatecache

Leading slash added to filenames under gulp 4

Open

#164 aberto em 28 de out. de 2018

Ver no GitHub
 (18 comments) (5 reactions) (0 assignees)JavaScript (97 forks)batch import
help wanted

Métricas do repositório

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

Description

Calling this package with gulp 4 erroneously results in template filenames with leading separators.

This seems to be due to the breaking change in vinyl, where internal path records such as file.base are now normalized: https://github.com/gulpjs/vinyl/commit/9ae7cd10c5423cbad9d38fc6c99960c62ac2f429#diff-c49ce3c4dfcfa830e06789881dd4d5d4 That change breaks the following code: url = path.join(root, file.path.replace(base || file.base, ''));

It is the root cause of #153

I posted a workaround in the other issue, but I'll see if I can cook something up that's compatible with all vinyl versions instead

Guia do colaborador

Direção de pesquisa
Inspecione o código em torno de `url = path.join(root, file.path.replace(base || file.base, ''))` e teste com diferentes versões do vinyl para garantir compatibilidade.
Pilha de tecnologia
javascriptnodejs
Domain
build systemtooling
Tipo Issue
Bug
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.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaScriptNode.jsgulp
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60