dev/lister_versions #46

Open
bricebou wants to merge 11 commits from dev/lister_versions into master
Collaborator

Première tentative de remise en ordre de marche de la commande core:listerversions suite à #45.

Première tentative de remise en ordre de marche de la commande core:listerversions suite à #45.
bricebou added 3 commits 7 months ago
rastapopoulos reviewed 7 months ago
rastapopoulos left a comment
Collaborator

Je trouve bizarre ce git init qui force un .git là où on se trouve. Lister les versions doit servir à pouvoir connaitre les versions existantes AVANT de télécharger, donc en étant depuis n'importe où, par ex dans le dossier parent ("tousmessites/"), et ya pas à créer un git du tout à cette endroit là.

Il me semble qu'avec les commandes de remote, on doit pouvoir préciser l'adresse du serveur dans l'appel de la commande non ?

https://git-scm.com/docs/git-ls-remote.html

<repository>
The "remote" repository to query. This parameter can be either a URL or the name of a remote

Je trouve bizarre ce git init qui force un .git là où on se trouve. Lister les versions doit servir à pouvoir connaitre les versions existantes AVANT de télécharger, donc en étant depuis n'importe où, par ex dans le dossier parent ("tousmessites/"), et ya pas à créer un git du tout à cette endroit là. Il me semble qu'avec les commandes de remote, on doit pouvoir préciser l'adresse du serveur *dans l'appel de la commande* non ? https://git-scm.com/docs/git-ls-remote.html > `<repository>` > The "remote" repository to query. This parameter can be either a URL or the name of a remote
Poster
Collaborator

Ah oui, en effet ! J'avais fait un test directement avec une URL mais j'avais obtenu une erreur (peut-être avais-je omis le .git).

Je corrige cela !

Après, je crois qu'il y a d'autres soucis de conception...

Ah oui, en effet ! J'avais fait un test directement avec une URL mais j'avais obtenu une erreur (peut-être avais-je omis le .git). Je corrige cela ! Après, je crois qu'il y a d'autres soucis de conception...
bricebou added 1 commit 7 months ago
Poster
Collaborator

En fait, en réfléchissant un peu plus, pas mal de questions se présentent, dont :

  • on devrait toujours lister par défaut les branches non sauf si derrière spip core:telecharger prend en charge les tags comme argument ?
  • la fonction execute() ne devrait-elle pas appeler que listerVersions, sans s'encombrer de vouloir définir la dernière et l'avant-dernière version en X.Y ? à moins que l'on se serve de la dernière version pour remplacer la constante _BRANCHE_STABLE de CoreTelecharger ?
En fait, en réfléchissant un peu plus, pas mal de questions se présentent, dont : - on devrait toujours lister par défaut les branches non sauf si derrière `spip core:telecharger` prend en charge les tags comme argument ? - la fonction `execute()` ne devrait-elle pas appeler que `listerVersions`, sans s'encombrer de vouloir définir la dernière et l'avant-dernière version en X.Y ? à moins que l'on se serve de la dernière version pour remplacer la constante `_BRANCHE_STABLE` de CoreTelecharger ?
bricebou added 3 commits 7 months ago
Poster
Collaborator

Bon... J'espère que c'est déjà plus propre que les premiers commits... Merci pour vos retours :)

Bon... J'espère que c'est déjà plus propre que les premiers commits... Merci pour vos retours :)
bricebou added 3 commits 7 months ago
bricebou added 1 commit 7 months ago
This pull request has changes conflicting with the target branch.
src/Command/CoreListerVersions.php
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.