Telliot: Замените все `fmt.Errorf` на` errors.Wrap` и `errors.Errorf`

Статус проблемы: 1. Открыт 2. Запущен 3. Отправлен 4. Готово

__Этот выпуск теперь имеет финансирование в размере 10,0779 TRB (281,17 долл. США по цене 27,9 долл. США / TRB) .__

• Если вы хотите поработать над этой проблемой, вы можете «начать работу» на странице сведений о проблеме Gitcoin .
  
  
  
  • Хотите скинуться? Добавьте сюда свой вклад.
  
  
  • Вопросов? Оформить заказ Справка Gitcoin или чат Gitcoin
  
  
  • $ 1052111,78 больше профинансированных работ по OSS доступно в Gitcoin Issue Explorer
  
  

@themandalore
Мне интересно поработать над этим вопросом, пригласите меня через Gitcoin

мой план работы - внести все изменения

@developerfred , отправлено. Вы наша космическая обезьяна в отношении тестирования всей этой штуки с gitcoin, так что дайте мне знать, если вам что-нибудь понадобится

@themandalore Спасибо, отлично!

Похоже, что в httpRetriever.go и logConfig.go (пакет утилит) все еще есть fmt.Errorf
@developerfred

@themandalore Я проверю здесь, спасибо, что

@themandalore сделано.

Выглядит хорошо, дайте мне знать, что делать дальше в отношении gitcoin

@themandalore Вы можете добавить меня к награде, когда я пытаюсь выразить интерес, я получаю сообщение об ошибке. Я сообщил команде и анализирую код Gitcoin, чтобы понять, почему.

мой пользователь gitcoin :

@developerfred , я ничего не могу сделать. Я свяжусь с ними, но если это займет больше одного или двух дней, просто вернитесь, и мы закроем его и вручную исправим.

@themandalore отлично, я тоже с ними разговариваю.
Они исправят это сейчас, тогда процесс заключается в том, чтобы одобрить меня, а затем заплатить.

@themandalore готово, теперь вам нужно одобрить меня на Gitcoin, чтобы я отправил PR.

Повторное открытие, поскольку я имел в виду кое-что еще.
Нам нужно использовать errors.Wrap , errors.Wrapf а также удалить слова заикания, чтобы улучшить сообщения журнала. Такие слова, как - can't, error, failed следует удалить, так как они будут повторяться в последнем сообщении журнала.

Вот несколько примеров необходимых изменений:

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)

Обратите внимание на удаление слова ошибки - оно нам не нужно, так как оно ясно, и в окончательном сообщении печати оно будет включено дважды.

Использование errors.Wrap позволяет нам выполнять сопоставление ошибок в вызывающей программе и, как правило, более структурировано:
см. https://blog.golang.org/go1.13-errors. В нем упоминалось об использовании% w, но я не фанат, так как это весьма подвержено ошибкам. - легко использовать% v и трудно следовать в PR, поэтому я предпочитаю использовать метод wrap.

В вызывающей программе мы можем сделать что-то вроде if err == ErrNotFound (сопоставление ошибок по типу), что более сложно без использования метода переноса.

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

обратите внимание, что удаление слова failed не требуется. Также здесь мы не используем Wrapf , а просто Wrap

Статус проблемы: 1. Открыт 2. Запущен 3. Отправлен 4. Готово

__Работа начата__.

Каждый из этих пользователей заявил, что сможет завершить работу к 266 годам.
Пожалуйста, ознакомьтесь с их планами действий ниже:

1) developerfred был одобрен для начала работы.

Я бы с удовольствием поработал над этим вопросом, мой план работы - произвести все замены
2) Янус подал заявку на начало работы _ (Только утвердить работника | отклонить работника ) _.

Раньше я работал над проектами Go, и у меня также есть хорошие навыки поиска и сопоставления слов в Linux. На выполнение этой задачи у меня уйдет всего несколько часов.
3) zyfrank подал заявку на начало работы _ (Только одобрить работника | отклонить работника ) _.

Я знаком с го, хочу взять на себя эту задачу
4) Вера подала заявку на начало работы _ (Только спонсоры: одобрить работника | отклонить работника ) _.

Я мог бы заменить их все вручную или автоматически.

Узнайте больше на странице сведений о проблеме Gitcoin .

Статус проблемы: 1. Открыт 2. Запущен 3. Отправлен 4. Готово

__Работа на 10,0779 TRB (283,87 USD @ 28,16 USD / TRB) подана __:

• Узнайте больше на странице сведений о проблеме Gitcoin.
• Хотите скинуться? Добавьте сюда свой вклад.
• Вопросов? Оформить заказ Справка Gitcoin или чат Gitcoin
• На 1 125 841,61 долл. США больше профинансированных работ по OSS доступно в Gitcoin Issue Explorer