Telliot: Reemplaza todo `fmt.Errorf` por` errors.Wrap` y `errors.Errorf`

Estado del problema: 1. Abierto 2. Iniciado 3. Enviado 4. Listo

__Este número ahora tiene un financiamiento de 10.0779 TRB (281.17 USD @ $ 27.9 / TRB) adjunto .__

• Si desea trabajar en este problema, puede 'comenzar a trabajar' en la página de Detalles del problema de Gitcoin .
  
  
  
  • ¿Quieres contribuir? Agregue su propia contribución aquí .
  
  
  • ¿Preguntas? Consulte la ayuda de Gitcoin o el chat de Gitcoin
  
  
  • $ 1,052,111.78 más trabajo de OSS financiado disponible en Gitcoin Issue Explorer
  
  

@themandalore
Me interesa trabajar en este tema, invítame a través de Gitcoin

mi plan de trabajo es hacer todos los cambios

@developerfred , enviado. Eres nuestro mono espacial en lo que respecta a probar todo este asunto de gitcoin, así que avísame si necesitas algo.

@themandalore ¡ Gracias, perfecto!

Parece que todavía hay algo de fmt.Errorf en httpRetriever.go y logConfig.go (el paquete util)
@desarrolladorfred

@themandalore Lo comprobaré aquí, gracias por

@themandalore hecho.

Se ve bien, cuéntame los próximos pasos con respecto a gitcoin

@themandalore Puedes aparece un error. Informé al equipo y estoy analizando el código de Gitcoin para ver por qué.

mi usuario de gitcoin :

@developerfred , realmente no puedo hacer nada. Me comunicaré con ellos, pero si toma más de uno o dos días, simplemente vuelva a extender la mano y lo cerraremos y lo arreglaremos manualmente.

@themandalore perfecto, también estoy hablando con ellos.
Lo arreglarán ahora, luego el proceso es solo para aprobarme y luego pagar.

@themandalore hecho, ahora necesitas aprobarme en Gitcoin, para que envíe PR.

Reabrir ya que tenía algo más en mente.
Necesitamos utilizar errors.Wrap , errors.Wrapf y también eliminar las palabras entrecortadas para mejorar los mensajes de registro. Palabras como - can't, error, failed deben eliminarse ya que se repetirán en el mensaje de registro final.

A continuación, se muestran algunos ejemplos de cambios necesarios:

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)

Tenga en cuenta la eliminación de la palabra de error: no la necesitamos, ya que está clara y el mensaje de impresión final la incluirá dos veces.

El uso de errors.Wrap nos permite hacer una coincidencia de errores en la persona que llama y generalmente es más estructurado:
ver: https://blog.golang.org/go1.13-errors , mencionó el uso de% w, pero no soy un fan ya que es bastante propenso a errores. - es fácil de usar% v y difícil de seguir en las relaciones públicas, por lo que prefiero usar el método de ajuste.

En el llamador, podemos hacer algo como if err == ErrNotFound (coincidencia de error por tipo) que es más difícil sin usar el método de ajuste.

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

tenga en cuenta de nuevo la eliminación de la palabra failed no es necesario. Además, aquí no usamos Wrapf , sino solo Wrap

Estado del problema: 1. Abierto 2. Iniciado 3. Enviado 4. Listo

__Se ha iniciado el trabajo__.

Cada uno de estos usuarios afirmó que puede completar el trabajo en 266 años a partir de ahora.
Revise sus planes de acción a continuación:

1) developerfred ha sido aprobado para comenzar a trabajar.

Me encantaría trabajar en este tema, mi plan de trabajo es hacer todos los reemplazos
2) janus ha solicitado comenzar a trabajar _ (solo patrocinadores: aprobar trabajador | rechazar trabajador ) _.

He trabajado en proyectos de Go antes y también tengo buenas habilidades de linux para encontrar y unir palabras. Me tomaría solo unas horas hacer esta tarea.
3) zyfrank ha solicitado comenzar a trabajar _ (solo para financiadores: aprobar trabajador | rechazar trabajador ) _.

Estoy familiarizado con go, me gustaría tomar esta tarea.
4) la fe se ha aplicado para comenzar a trabajar _ (solo patrocinadores: aprueban al trabajador | rechazan al trabajador ) _.

Podría reemplazarlos todos de forma manual o automática.

Obtenga más información en la página de detalles del problema de Gitcoin .

Estado del problema: 1. Abierto 2. Iniciado 3. Enviado 4. Listo

__Trabajo para 10.0779 TRB (283.87 USD @ $ 28.16 / TRB) ha sido enviado por__:

• Obtenga más información en la página de detalles del problema de Gitcoin
• ¿Quieres contribuir? Agregue su propia contribución aquí .
• ¿Preguntas? Consulte la ayuda de Gitcoin o el chat de Gitcoin
• $ 1,125,841.61 más trabajo de OSS financiado disponible en Gitcoin Issue Explorer