Telliot: Remplacez tous les `fmt.Errorf` par `errors.Wrap` et `errors.Errorf`

État du problème : 1. Ouvert 2. Commencé 3. Soumis 4. Terminé

__Ce numéro est désormais associé à un financement de 10,0779 TRB (281,17 USD à 27,9 $/TRB).__

• Si vous souhaitez travailler sur ce problème, vous pouvez « commencer à travailler » sur la page Détails du problème Gitcoin .
  
  
  
  • Vous voulez participer? Ajoutez votre propre contribution ici .
  
  
  • Des questions? Commander l' aide Gitcoin ou le chat Gitcoin
  
  
  • 1 052 111,78 $ de plus de travaux OSS financés disponibles sur l' explorateur de problèmes Gitcoin
  
  

@themandalore
Je suis intéressé à travailler sur ce problème, invitez-moi via Gitcoin

mon plan de travail est de faire tous les changements

@developerfred , envoyé. Vous êtes notre singe de l'espace en ce qui concerne le test de tout ce truc de gitcoin, alors faites-moi savoir si vous avez besoin de quelque chose

@themandalore Merci, parfait !

On dirait qu'il y a encore des fmt.Errorf dans httpRetriever.go et logConfig.go (le package util)
@developpeurfred

@themandalore Je vais vérifier ici, merci de me l'avoir fait savoir.

@themandalore terminé.

Ça a l'air bien, faites-moi savoir les prochaines étapes en ce qui concerne le gitcoin

@themandalore Vous pouvez m'ajouter à la prime, lorsque j'essaie d'exprimer mon intérêt, j'obtiens une erreur. J'ai signalé à l'équipe et j'analyse le code Gitcoin pour voir pourquoi.

mon utilisateur gitcoin :

@developerfred , je ne peux vraiment rien faire. Je vais les contacter, mais si cela prend plus d'un jour ou deux, revenez simplement et nous le fermerons et le réparerons manuellement

@themandalore parfait, je leur parle aussi.
Ils vont le réparer maintenant, alors le processus consiste simplement à m'approuver et à payer.

@themandalore terminé, vous devez maintenant m'approuver sur Gitcoin pour que je puisse soumettre des relations publiques.

Réouverture car j'avais autre chose en tête.
Nous devons utiliser errors.Wrap , errors.Wrapf et également supprimer les mots saccadés pour améliorer les messages de journal. Les mots tels que - can't, error, failed doivent être supprimés car ils seront répétés dans le message de journal final.

Voici quelques exemples de modifications requises :

errors.Errorf("file %s stat error: %v", historyPath, err)
changed to
errors.Wrapf(err,"stats for file: %v", historyPath)
alternative - but I am not a big fan of this one as error prone.
errors.Errors("stats for file: %v, err:%w", historyPath,err)

Notez la suppression du mot d'erreur - nous n'en avons pas besoin car cela est clair et le message d'impression final l'inclura deux fois.

L'utilisation de errors.Wrap nous permet de faire une correspondance d'erreur dans l'appelant et c'est généralement plus structuré :
voir : https://blog.golang.org/go1.13-errors , Il a mentionné l'utilisation de %w, mais je ne suis pas un fan car il est assez sujet aux erreurs. - il est facile à utiliser %v et difficile à suivre dans les PR donc je préfère utiliser la méthode wrap.

Dans l'appelant, nous pouvons faire quelque chose comme if err == ErrNotFound (erreur de correspondance par type) qui est plus difficile sans utiliser la méthode wrap.

errors.Errorf("failed to read psr file @ %s: %v", historyPath, err)
changed to
errors.Wrap(err,"read psr file:%v", historyPath)

s'il vous plaît noter à nouveau la suppression du mot failed il n'est pas nécessaire. Ici aussi, nous n'utilisons pas Wrapf , mais juste Wrap

État du problème : 1. Ouvert 2. Commencé 3. Soumis 4. Terminé

__Les travaux ont commencé__.

Ces utilisateurs ont chacun affirmé qu'ils pouvaient terminer le travail d'ici 266 ans.
Veuillez consulter leurs plans d'action ci-dessous :

1) developerfred a été approuvé pour commencer le travail.

J'adorerais travailler sur cette question, mon plan de travail est de faire tous les remplacements
2) Janus a fait une demande pour commencer le travail _(Bailleurs de fonds uniquement : approuver le travailleur | rejeter le travailleur )_.

J'ai déjà travaillé sur des projets Go et j'ai également de bonnes compétences Linux pour trouver et faire correspondre des mots. Je ne prendrais que quelques heures pour accomplir cette tâche.
3) zyfrank a postulé pour commencer le travail _(Bailleurs de fonds uniquement : approuver le travailleur | rejeter le travailleur )_.

Je connais le go, j'aimerais assumer cette tâche
4) la foi s'est appliquée pour commencer le travail _(Bailleurs de fonds uniquement : approuver le travailleur | rejeter le travailleur )_.

Je pourrais tous les remplacer manuellement ou automatiquement.

En savoir plus sur la page Détails du problème Gitcoin .

État du problème : 1. Ouvert 2. Commencé 3. Soumis 4. Terminé

__Le travail pour 10,0779 TRB (283,87 USD @ 28,16 $/TRB) a été soumis par__ :

• En savoir plus sur la page Détails du problème Gitcoin
• Vous voulez participer? Ajoutez votre propre contribution ici .
• Des questions? Commander l' aide Gitcoin ou le chat Gitcoin
• 1 125 841,61 $ de plus de travaux OSS financés disponibles sur l' explorateur de problèmes Gitcoin