Telliot: 어디서나 printf 대신 일관된 로깅을 사용하십시오.

오늘 이거 먹겠습니다!

업데이트 : 이것은 부분적으로 수행되었지만 여전히 printf 및 logrus가있는 곳이 거의 없습니다.
TL-DR은 모든 곳에서 github.com/go-kit/kit/log를 사용해야합니다. 구성 요소를 초기화 할 때 인수로 전달해야합니다.

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

__ 이번 호에는 Tellor-io 펀드의 일부로 0.1 ETH (114.26 USD @ $ 1142.63 / ETH)의 자금이 첨부되어 있습니다 .__

• 이 문제를 해결 하고 싶다면 Gitcoin Issue Details 페이지에서 '작업을 시작'할 수 있습니다.
  
  
  
  • 칩을 원하십니까? 여기에 자신의 기여를 추가
  
  
  • 질문이 있으십니까? Gitcoin 도움말 또는 Gitcoin의 Discord 확인
  
  
  • Gitcoin Issue Explorer 에서 $ 1,033,707.71 추가 자금 지원 OSS 작업 가능
  
  

@ krasi-georgiev 나는 이것을 취하고 싶지만 Gitcoin에 대한 귀하의 제안을 읽고 Telliot 코드를 읽은 후 모든 주석 처리 된 fmt.Printf를 level.Debug (logger) .Log (... .) 우리는 사용자 관점에서 더 의미있는 것이 무엇인지 각각의 상황을 결정해야합니다.
출력이 개발자 관점에서 문제를 검사하는 데 유용해야 할 때 로그 디버그를 사용하지만 사용자가 콘솔에 출력을 기대하는 상황은 INFO 수준이어야합니다. 예를 들어
https://github.com/tellor-io/telliot/blob/master/pkg/ops/disputeOps.go#L249 -L255

    fmt.Printf("Dispute %s (%s):\n", dispute.DisputeId.String(), descString)
    fmt.Printf("    Accused Party: %s\n", reportedAddr.Hex())
    fmt.Printf("    Disputed by: %s\n", reportingMiner.Hex())
    fmt.Printf("    Created on:  %s\n", createdTime.Format("3:04 PM January 02, 2006 MST"))
    fmt.Printf("    Fee: %s TRB\n", util.FormatERC20Balance(uintVars[8]))
    fmt.Printf("    \n")
    fmt.Printf("    Value disputed for requestID %d:\n", dispute.RequestId.Uint64())

내가 틀렸다면 나를 고쳐주세요. 감사!

네, 매우 사실입니다! 또 다른 규칙은 로그가 시작될 때 한 번 발생한다는 것입니다. 정보가되는 것은 좋지만, 대부분의 경우 매우 중요하지 않는 한 루프에서 여러 번 발생하면 이것은 너무 많은 노이즈이며 디버그해야합니다.

@ krasi-georgiev 예, 사용자를 압도하지 않는 것이 합리적이지만 도구를 사용하여 로그를 분석하려는 상황에서는 분석을위한 유일한 피드백이기 때문에 동작 변경이 혼란 스러울 수 있다고 생각합니다. 우리는 제 생각에 올바른 균형을 얻을 것입니다. 여기서 몇 가지 변경 사항을 수행 할 것이며이 프로젝트에 대해 더 많이 배우면 좋은 사용자 경험에 적합한 것이 무엇인지 찾을 것입니다.

이 작업이 완료되면 동일한 PR에 다음과 같은 makefile에 더 많은 linting을 추가해야합니다.

go-lint: check-git deps $(GOLANGCI_LINT) $(FAILLINT)
    $(call require_clean_work_tree,'detected not clean master before running lint, previous job changed something?')
    <strong i="6">@echo</strong> ">> verifying modules being imported"
    @$(FAILLINT) -paths "errors=github.com/pkg/errors" ./...
    @$(FAILLINT) -paths "fmt.{Print,Printf,Println,Sprint}" -ignore-tests ./...
    <strong i="7">@echo</strong> ">> linting all of the Go files GOGC=${GOGC}"
    @$(GOLANGCI_LINT) run
    <strong i="8">@echo</strong> ">> ensuring Copyright headers"
    <strong i="9">@go</strong> run ./scripts/copyright
    $(call require_clean_work_tree,'detected files without copyright, run make lint and commit changes')

2 개의 etra 라인은 다음과 같습니다.

    @$(FAILLINT) -paths "errors=github.com/pkg/errors" ./...
    @$(FAILLINT) -paths "fmt.{Print,Printf,Println,Sprint}" -ignore-tests ./...

이것은 어디에도 print 문이 없으며 pkg/errors 가 어디에서나 사용되도록 강제합니다.

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

__ 작업이 시작되었습니다 __.

이 사용자들은 각각 6 개월 후 265 년까지 작업을 완료 할 수 있다고 주장했습니다.
아래에서 그들의 행동 계획을 검토하십시오.

1) g33kidd 가 작업 시작을 신청했습니다 _ (펀더 만 해당 : 작업자 승인 | 작업자 거부 ) _.

나는 이것을 찔러 볼 것이다. 그것은 흥미로운 일인 것 같았다.
2) voanhcung 이 작업 시작을 신청했습니다 _ (자금 만 해당 : 작업자 승인 | 작업자 거부 ) _.

Ok [email protected] .......................
3) therocketcat 이 작업을 시작하도록 승인되었습니다.

코드베이스와 앞서 언급 한 예제 프로젝트를 살펴 보았습니다.

예제에서 printf를 사용하는 함수에 생성자가 없으면 로거가 함수 자체에 종속성을 주입해야한다는 것을 수집했습니다. 이것이 사실이 아니라면 거기에 설명하고 싶습니다.

그리고 기본 패키지 (telliot / cmd / telliot / main.go)에서는 util의 SetupLogger 함수를 사용하여 설정되고 생성자에 전달되거나 함수에 직접 삽입됩니다 (첫 번째 매개 변수로).

또한 코드베이스를 보면 주석 처리 된 printfs를 찾을 수 없으므로 일반적으로 printfs를 의미한다고 가정합니까?

RocketCat # 3507 작업을 오해 한 경우이 문제를 추가로 논의 할 수 있습니다.
4) mendesfabio 가 작업 시작을 신청했습니다 _ (자금 만 해당 : 작업자 승인 | 작업자 거부 ) _.

방금 repo를 확인했는데 일부 printf-s가 교체 된 것 같습니다 (그리고이를 수행하는 방법을 이해하는 데 도움이되었습니다). 우리는 여전히 printf-s가있는 10 개의 파일을 가지고 있으며 그것을 변경할 것입니다.
5) rodrigoadriandiaz 가 작업 시작을 신청했습니다 _ (자금 만 제공 : 작업자 승인 | 작업자 거부 ) _.

위에서 언급 한 로깅 패키지의 구현을 위해 printf에 대한 모든 호출을 대체하여이 작업을 수행 할 수 있습니다. 내일이 작업을 시작할 수 있습니다.
6) raulcorreia7 이 작업 시작을 신청했습니다 _ (자금 만 제공 : 작업자 승인 | 작업자 거부 ) _.

안녕하세요,
이것은 생각할 필요가 없습니다.
DM 나.
7) coder4520 이 작업 시작을 신청했습니다 _ (펀더 만 해당 : 작업자 승인 | 작업자 거부 ) _.

처음 시작하기에 정말 좋은 프로젝트라고 생각합니다.
8) adiprerepa 가 작업 시작을 신청했습니다 _ (자금 만 제공 : 작업자 승인 | 작업자 거부 ) _.

이것은 매우 쉬울 것입니다. printf를 찾아서 로깅으로 바꾼 다음 다시 확인하십시오.
9) xf3rno 가 작업 시작에 적용되었습니다 _ (펀더 만 해당 : 작업자 승인 | 작업자 거부 ) _.

요청한 작업을 수행합니다. 모든 현재 로깅을 Google의 golang 로거 라이브러리 (매우 경량 : https://github.com/google/logger)로 바꾸고 컨텍스트 (디버그, 경고 등)에 따라 각 메시지의 범위를 지정합니다.

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

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

__0.1 ETH (167.47 USD @ $ 1674.72 / ETH)에 대한 작업이 __에 의해 제출되었습니다.

• Gitcoin 문제 세부 정보 페이지에서 자세히 알아보기
• 칩을 원하십니까? 여기에 자신의 기여를 추가
• 질문이 있으십니까? Gitcoin 도움말 또는 Gitcoin의 Discord 확인
• Gitcoin Issue Explorer 에서 $ 1,072,039.97 추가 자금 지원 OSS 작업 가능