Telliot: Ersetzen Sie alle `fmt.Errorf` durch `errors.Wrap` und `errors.Errorf`

Problemstatus: 1. Offen 2. Gestartet 3. Gesendet 4. Fertig

__Diese Ausgabe ist jetzt mit einer Finanzierung von 10,0779 TRB (281,17 USD @ 27,9/TRB) verbunden.__

• Wenn Sie an diesem Problem arbeiten möchten, können Sie
  
  
  
  • Willst du mitmachen? Fügen Sie hier Ihren eigenen Beitrag hinzu.
  
  
  • Fragen? Schauen Sie sich die Gitcoin- Hilfe oder den Gitcoin-Chat an
  
  
  • 1.052.111,78 $ mehr finanzierte OSS-Arbeit im Gitcoin Issue Explorer verfügbar
  
  

@themandalore
Ich bin daran interessiert, an diesem Thema zu arbeiten, lade mich über Gitcoin ein

Mein Arbeitsplan ist es, alle Änderungen vorzunehmen

@developerfred , gesendet. Du bist unser Weltraumaffe, wenn es darum geht, diese ganze Gitcoin-Sache zu testen, also lass es mich wissen, wenn du etwas brauchst

@themandalore Danke, perfekt!

Es sieht so aus, als ob in httpRetriever.go und logConfig.go (dem util-Paket) immer noch fmt.Errorf vorhanden sind.
@entwicklerfred

@themandalore Ich werde hier Information .

@themandalore fertig.

Sieht gut aus, lass mich die nächsten Schritte in Bezug auf gitcoin wissen

@themandalore Sie können mich zum Kopfgeld hinzufügen, wenn ich versuche, Interesse zu bekunden, erhalte ich eine Fehlermeldung. Ich habe dem Team Bericht erstattet und analysiere den Gitcoin-Code, um herauszufinden, warum.

mein gitcoin-Benutzer: @developerfred

@developerfred , ich kann nicht wirklich etwas tun. Ich werde mich an sie wenden, aber wenn es länger als ein oder zwei Tage dauert, melden Sie sich einfach wieder und wir schließen es und beheben es manuell

@themandalore perfekt, ich rede auch mit ihnen.
Sie werden es jetzt reparieren, dann besteht der Prozess nur darin, mich zu genehmigen und dann zu bezahlen.

@themandalore fertig, jetzt müssen Sie mich auf Gitcoin genehmigen, damit ich PR einreichen kann.

Wiedereröffnung, da ich etwas anderes im Sinn hatte.
Wir müssen errors.Wrap , errors.Wrapf und auch die Stotterwörter entfernen, um die Protokollmeldungen zu verbessern. Wörter wie - can't, error, failed sollten entfernt werden, da diese in der letzten Protokollnachricht wiederholt werden.

Hier einige Beispiele für erforderliche Änderungen:

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)

Beachten Sie, dass Sie das Fehlerwort entfernen - wir brauchen es nicht, da dies klar ist und die endgültige Drucknachricht es zweimal enthält.

Die Verwendung von errors.Wrap ermöglicht uns eine Fehlersuche im Aufrufer und ist im Allgemeinen strukturierter:
siehe: https://blog.golang.org/go1.13-errors , Es wurde die Verwendung von %w erwähnt, aber ich bin kein Fan, da es ziemlich fehleranfällig ist. - Es ist einfach, %v zu verwenden und in PRs schwer zu folgen, daher bevorzuge ich die Wrap-Methode.

Im Aufrufer können wir etwas wie if err == ErrNotFound (Fehlerübereinstimmung nach Typ) tun, was ohne die Wrap-Methode schwieriger ist.

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

Bitte beachten Sie erneut, dass das Wort failed Es wird nicht benötigt. Auch hier verwenden wir nicht Wrapf , sondern nur Wrap

Problemstatus: 1. Offen 2. Gestartet 3. Gesendet 4. Fertig

__Arbeit wurde begonnen__.

Diese Benutzer gaben jeweils an, dass sie die Arbeit in 266 Jahren abschließen können.
Bitte überprüfen Sie ihre Aktionspläne unten:

1) Developerfred wurde zur Arbeitsaufnahme freigegeben.

Ich würde gerne an diesem Problem arbeiten, mein Arbeitsplan ist es, alle Ersetzungen vorzunehmen
2) Janus hat die Aufnahme der Arbeit beantragt _(nur Geldgeber: Arbeiter genehmigen | Arbeiter ablehnen )_.

Ich habe bereits an Go-Projekten gearbeitet und habe auch gute Linux-Kenntnisse, um Wörter zu finden und zuzuordnen. Ich würde nur Stunden brauchen, um diese Aufgabe zu erledigen.
3) zyfrank hat einen Antrag auf Aufnahme der Arbeit gestellt _(nur Arbeiter genehmigen | Arbeiter ablehnen )_.

Ich kenne go, diese Aufgabe würde ich gerne übernehmen
4) Glaube hat sich für die Aufnahme der Arbeit beworben _(nur Geldgeber: Arbeiter genehmigen | Arbeiter ablehnen )_.

Ich könnte sie alle manuell oder automatisch ersetzen.

Weitere Informationen finden Sie

Problemstatus: 1. Offen 2. Gestartet 3. Gesendet 4. Fertig

__Arbeit für 10,0779 TRB (283,87 USD @ 28,16/TRB) wurde eingereicht von__:

• Erfahren Sie mehr auf der Seite mit den Details zu Gitcoin-Problemen
• Willst du mitmachen? Fügen Sie hier Ihren eigenen Beitrag hinzu.
• Fragen? Schauen Sie sich die Gitcoin- Hilfe oder den Gitcoin-Chat an
• 1.125.841,61 $ mehr finanzierte OSS-Arbeit im Gitcoin Issue Explorer verfügbar