miickel/gulp-angular-templatecache

Leading slash added to filenames under gulp 4

Open

#164 ouverte le 28 oct. 2018

Voir sur GitHub
 (18 commentaires) (5 réactions) (0 assignés)JavaScript (97 forks)batch import
help wanted

Métriques du dépôt

Stars
 (533 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

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

Guide contributeur

Direction de recherche
Inspectez le code autour de `url = path.join(root, file.path.replace(base || file.base, ''))` et testez avec différentes versions de vinyl pour assurer la compatibilité.
Stack technique
javascriptnodejs
Domaine
build systemtooling
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScriptNode.jsgulp
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60