ipfs/kubo

Rename "ipfs swarm peers" to "ipfs swarm conns"

Open

#4 196 ouverte le 1 sept. 2017

Voir sur GitHub
 (4 commentaires) (0 réactions) (0 assignés)Go (2 725 forks)batch import
help wantedkind/enhancement

Métriques du dépôt

Stars
 (13 906 stars)
Métriques de merge PR
 (Merge moyen 4j 2h) (28 PRs mergées en 30 j)

Description

Type:

Enhancement(ish)

Severity:

Pedantic

Description:

ipfs swarm peers is misnamed, it should be ipfs swarm conns (it prints connections). We should:

  1. Introduce the new command.
  2. Deprecate the old one and hide it (but don't actually remove it).

Supersedes #4104 (would break too much)

Guide contributeur

Direction de recherche
Le problème nécessite l'ajout d'une nouvelle commande 'ipfs swarm conns' qui aliasera la commande existante 'ipfs swarm peers', puis la dépréciation et le masquage de l'ancienne commande. Le dépôt utilise Go et la CLI est construite avec cobra. Examinez l'implémentation existante de 'ipfs swarm peers' (probablement dans core/commands/swarm.go) pour comprendre la structure. La nouvelle commande doit appeler le même gestionnaire. Suivez le modèle de dépréciation utilisé ailleurs dans la base de code, par exemple en marquant l'ancienne commande comme cachée et en ajoutant un message de dépréciation. Aucune PR ou commentaire lié n'indique un travail antérieur, il s'agit donc d'un territoire nouveau.
Stack technique
go
Domaine
backendcli
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
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
GoGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
90