Telliot: 모든 `fmt.Errorf`를 `errors.Wrap` 및 `errors.Errorf`로 바꿉니다.

문제 상태 : 3. 시작 1. 2. 완료 4. 제출

__이 호에는 현재 10.0779 TRB(281.17 USD @ $27.9/TRB)의 자금이 첨부되어 있습니다.__

• 이 문제에 대해 작업 하고 싶다면 Gitcoin 문제 세부 정보 페이지에서 '작업 시작'할 수 있습니다.
  
  
  
  • 칩 인을 원하십니까? 여기에 자신의 기여를 추가
  
  
  • 질문? Gitcoin 도움말 또는 Gitcoin 채팅 확인
  
  
  • Gitcoin Issue Explorer 에서
  
  

@themandalore
이 문제에 관심이 있습니다. Gitcoin을 통해 저를 초대하십시오.

내 작업 계획은 모든 변경 사항을 만드는 것입니다

@developerfred , 보냈습니다. 이 전체 gitcoin을 테스트하는 것과 관련하여 당신은 우리의 우주 원숭이입니다. 따라서 필요한 것이 있으면 알려주십시오.

@themandalore 감사합니다, 완벽합니다!

httpRetriever.go 및 logConfig.go(util 패키지)에 여전히 fmt.Errorf가 있는 것 같습니다.
@developerfred

@themandalore 여기서 확인하겠습니다. 알려주셔서 감사합니다.

@themandalore 완료.

좋아 보입니다. gitcoin과 관련된 다음 단계를 알려주세요.

@themandalore 저를 현상금에 추가할 수 있습니다. 관심을 표현하려고 하면 오류가 발생합니다. 팀에 보고했고 이유를 알아보기 위해 Gitcoin 코드를 분석하고 있습니다.

내 gitcoin 사용자: @developerfred

@developerfred , 나는 정말로 아무것도 할 수 없습니다. 연락을 드리겠습니다. 하지만 하루나 이틀 이상 소요되는 경우 다시 연락해 주세요. 닫고 수동으로 수정하겠습니다.

@themandalore는 완벽합니다. 저도 그들과 이야기하고 있습니다.
그들은 지금 그것을 고칠 것이고, 그 과정은 단지 나를 승인하고 지불하는 것입니다.

@themandalore 완료, 이제 PR을 제출하려면 Gitcoin에서 저를 승인해야 합니다.

다른 생각이 있어서 다시 열었습니다.
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) janus 가 작업 시작을 신청했습니다_(Funders만: 승인 작업자 | 거부 작업자 )_.

나는 이전에 Go 프로젝트에서 일한 적이 있으며 단어를 찾고 일치시키는 훌륭한 Linux 기술도 가지고 있습니다. 이 작업을 완료하는 데 몇 시간 밖에 걸리지 않습니다.
3) zyfrank 가 작업 시작_(Funders only: 승인 작업자 | 거부 작업자 )_을 신청했습니다.

나는 go에 익숙하다, 나는 이 일을 하고 싶다
4) 믿음 이 일을 시작하기 위해 적용되었습니다_(후원자만: 승인 작업자 | 거부 작업자 )_.

수동 또는 자동으로 모두 교체할 수 있습니다.

Gitcoin 문제 세부정보 페이지에서 자세히 알아보세요.

문제 상태: 1. 진행 중 2. 시작됨 3. 제출됨 4. 완료됨

__10.0779 TRB(283.87 USD @ $28.16/TRB)에 대한 작업이 __에 의해 제출되었습니다.

• Gitcoin 문제 세부 정보 페이지에서 자세히 알아보기
• 칩 인을 원하십니까? 여기에 자신의 기여를 추가
• 질문? Gitcoin 도움말 또는 Gitcoin 채팅 확인
• Gitcoin Issue Explorer 에서