Yarn: Impossible d'installer une branche avec un # dans le nom avec git + ssh

Créé le 8 juin 2018  ·  3Commentaires  ·  Source: yarnpkg/yarn

Voulez-vous demander une fonctionnalité ou signaler un bogue ?
Ceci est un rapport de bogue.

Quel est le comportement actuel?
Lors de l'exécution de yarn install lorsque package.json contient une dépendance qui utilise une URL git+ssh et
spécifie un nom de branche avec un # dans le nom (courant pour les problèmes de suivi) l'installation échoue avec l'erreur suivante:

error Couldn't find match for "issue-" in "refs/heads/issue-#1,refs/heads/master" for "[email protected]:cdimitroulas/test-repository.git".

Exemple package.json dépendance:

"dependencies": {
    "test-module": "git+ssh://[email protected]:cdimitroulas/test-repository.git#issue-#1"
  }

Si le comportement actuel est un bogue, veuillez fournir les étapes à reproduire.

  1. Créez un nouveau dépôt et initialisez avec npm init -y .
  2. Créez le package.json :
{
  "name": "yarn-bug",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "keywords": [],
  "author": "",
  "license": "ISC",
  "dependencies": {
    "test-module": "git+ssh://[email protected]:cdimitroulas/test-repository.git#issue-#1"
  }
}
  1. Exécutez yarn install

Quel est le comportement attendu?
yarn install devrait installer mon module à partir de la branche spécifiée.

Veuillez mentionner votre node.js, votre fil et la version de votre système d'exploitation.
npm - 6.1.0
fil - 1.7.0
os - Ubuntu 16.04 (x64)

cat-bug

Tous les 3 commentaires

Confirmé sur v1.7.0. Je vais voir si je peux créer un PR pour ça aujourd'hui ...

Génial, merci beaucoup pour le rapide PR @ rally25rs : heart_eyes:

J'ai découvert qu'il s'agissait d'un duplicata de # 4880 qui avait déjà un autre PR # 4881 ouvert pour le réparer. Je vais fermer ce problème en double et fermer mon PR car c'est le même que celui qui existait déjà. Il y a une discussion dans # 4881 pour savoir si nous devrions ou non prendre en charge cette syntaxe ...

Cette page vous a été utile?
0 / 5 - 0 notes