Telliot: हर जगह प्रिंटफ की जगह कंसिस्टेंट लॉगिंग का इस्तेमाल करें।

मैं आज इसे लूंगा!

एक अद्यतन: यह आंशिक रूप से किया जाता है, लेकिन अभी भी कुछ स्थान प्रिंटफ और लॉगरस के साथ हैं।
TL-DR को हर जगह github.com/go-kit/kit/log का उपयोग करने की आवश्यकता है। घटकों को प्रारंभ करते समय इसे तर्क के रूप में पारित करने की आवश्यकता होती है।

समस्या की स्थिति: १. खुला २. शुरू हुआ ३. सबमिट किया गया ४. हो गया

__इस इश्यू में अब 0.1 ETH (114.26 USD @ 1142.63/ETH) की फंडिंग टेलर-आईओ फंड के हिस्से के रूप में जुड़ी हुई है।__

• यदि आप इस मुद्दे पर काम करना चाहते हैं तो आप Gitcoin समस्या विवरण पृष्ठ पर 'काम शुरू' कर सकते हैं।
  
  
  
  • चिप लगाना चाहते हैं? यहां अपना योगदान जोड़ें।
  
  
  • प्रशन? चेकआउट गिटकोइन सहायता या गिटकोइन का विवाद
  
  
  • $1,033,707.71 अधिक वित्त पोषित OSS कार्य Gitcoin समस्या एक्सप्लोरर पर उपलब्ध है
  
  

@ krasi-georgiev मैं इसे लेना चाहता हूं लेकिन मैं गिटकोइन पर आपके सुझावों को पढ़ने और टेलियट कोड पढ़ने के बाद कहूंगा कि सभी को बदलने के बजाय fmt.Printf को स्तर पर बदलें। डीबग (लॉगर)। लॉग (... ।) हमें प्रत्येक स्थिति पर निर्णय लेना चाहिए कि उपयोगकर्ता के दृष्टिकोण से क्या अधिक समझ में आता है।
मैं लॉग डीबग का उपयोग तब करूंगा जब आउटपुट किसी समस्या का निरीक्षण करने के लिए उपयोगी होना चाहिए, डेवलपर के दृष्टिकोण से अधिक, लेकिन ऐसी स्थितियां जहां उपयोगकर्ता कंसोल से आउटपुट की अपेक्षा करता है, जानकारी स्तर होना चाहिए। जैसे उदहारण के लिए
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 हां, उपयोगकर्ता को अभिभूत नहीं करने के लिए समझ में आता है, लेकिन मुझे लगता है कि ऐसी परिस्थितियों में जहां आप किसी टूल के साथ लॉग का विश्लेषण करना चाहते हैं, व्यवहार बदलना भ्रमित हो सकता है क्योंकि आपके विश्लेषण के लिए केवल यही प्रतिक्रिया है, लेकिन वैसे भी मुझे लगता है कि हमें सही संतुलन मिलेगा। मैं यहां कुछ बदलाव करने जा रहा हूं और जैसे-जैसे मैं इस परियोजना के बारे में और जानूंगा, हम पाएंगे कि एक अच्छे उपयोगकर्ता अनुभव के लिए क्या काम करता है।

एक बार ऐसा करने के बाद उसी पीआर में मेकफ़ाइल में कुछ और लाइनिंग जोड़नी चाहिए जैसे

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 एट्रा लाइनें हैं:

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

यह लागू करेगा कि कहीं भी कोई प्रिंट स्टेटमेंट नहीं है और यह भी कि pkg/errors का उपयोग कहीं भी किया जाता है

समस्या की स्थिति: 1. खुला 2. शुरू हुआ 3. सबमिट किया गया 4. हो गया

__काम शुरू कर दिया गया है__।

इन उपयोगकर्ताओं में से प्रत्येक ने दावा किया कि वे 265 साल, अब से 6 महीने बाद काम पूरा कर सकते हैं।
कृपया नीचे उनकी कार्य योजनाओं की समीक्षा करें:

१) g33kidd ने काम शुरू करने के लिए आवेदन किया है _(केवल फंडर्स: वर्कर | रिजेक्ट वर्कर )_।

मैं इस पर एक छुरा घोंपूंगा, ऐसा करना एक दिलचस्प बात लग रही थी।
2) वोनहकुंग ने काम शुरू करने के लिए आवेदन किया है _(केवल फंडर्स: वर्कर को मंजूरी दें | कार्यकर्ता को अस्वीकार करें )_।

ठीक है [email protected].................................
3) थेरॉकेटकैट को काम शुरू करने की मंजूरी मिल गई है।

मैंने आपके द्वारा उल्लिखित कोड आधार और पहले बताई गई उदाहरण परियोजनाओं को देखा।

उदाहरणों से मैंने इकट्ठा किया है कि यदि प्रिंटफ का उपयोग करने वाले कार्यों में कोई कन्स्ट्रक्टर नहीं है तो लॉगर को निर्भरता को फ़ंक्शन में ही इंजेक्ट किया जाना चाहिए। यदि ऐसा नहीं है, तो मैं वहां स्पष्टीकरण चाहता हूं।

और मुख्य पैकेज (telliot/cmd/telliot/main.go) में उन्हें उपयोग से SetupLogger फ़ंक्शन का उपयोग करके सेटअप किया जाएगा और या तो कंस्ट्रक्टर को पास किया जाएगा या सीधे फ़ंक्शन में इंजेक्ट किया जाएगा (पहले पैरामीटर के रूप में)।

कोडबेस को भी देखकर मुझे प्रिंटफ पर कोई टिप्पणी नहीं मिल रही है, इसलिए मुझे लगता है कि आप सामान्य रूप से प्रिंटफ का मतलब रखते हैं?

इस पर आगे चर्चा करने के लिए खोलें यदि मैंने कार्य को गलत समझा है RocketCat#3507
4) मेंडेसफैबियो ने काम शुरू करने के लिए आवेदन किया है _(केवल फंडर्स: वर्कर | रिजेक्ट वर्कर )_।

मैंने अभी रेपो की जांच की है और ऐसा लगता है कि कुछ प्रिंटफ-एस को बदल दिया गया था (और इससे मुझे यह समझने में मदद मिली कि इसे कैसे किया जाए)। हमारे पास अभी भी प्रिंटफ-एस के साथ 10 फाइलें हैं और मैं इसे बदल दूंगा।
५) रोड्रिगोअड्रियाज ने काम शुरू करने के लिए आवेदन किया है _(केवल फंडर्स: अप्रूव वर्कर | रिजेक्ट वर्कर )_।

मैं यह कर सकता था, उपर्युक्त लॉगिंग पैकेज के कार्यान्वयन के लिए सभी कॉलों को प्रिंटफ में बदलकर, मैं कल इस पर काम करना शुरू कर सकता हूं
६) raulcorreia7 ने काम शुरू करने के लिए आवेदन किया है _(केवल फंडर्स: वर्कर | रिजेक्ट वर्कर )_।

नमस्ते,
यह कोई ब्रेनर नहीं होगा,
मुझे डीएम।
7) coder4520 ने काम शुरू करने के लिए आवेदन किया है _(केवल फंडर्स: वर्कर | रिजेक्ट वर्कर )_।

मुझे लगता है कि शुरू करने के लिए यह वास्तव में पहली परियोजना होगी।
8) आदिप्रेपा ने काम शुरू करने के लिए आवेदन किया है _(केवल फंडर्स: वर्कर | रिजेक्ट वर्कर )_

यह बहुत आसान होगा - बस प्रिंटफ को लॉगिंग में ढूंढें और बदलें और फिर दोबारा जांच करें।
9) xf3rno ने काम शुरू करने के लिए आवेदन किया है _(केवल फंडर्स: वर्कर | रिजेक्ट वर्कर )_।

मैं अनुरोधित कार्य निष्पादित करूंगा; सभी मौजूदा लॉगिंग को Google की गोलांग लॉगर लाइब्रेरी (काफी हल्का: https://github.com/google/logger) से बदलें, और संदर्भ (डीबग, चेतावनी, आदि) के आधार पर प्रत्येक संदेश को स्कोप करें।

Gitcoin समस्या विवरण पृष्ठ पर और जानें।

समस्या की स्थिति: १. खुला २. प्रारंभ ३. सबमिट किया गया ४. हो गया

__0.1 ETH (167.47 USD @ $1674.72/ETH) के लिए कार्य__ द्वारा प्रस्तुत किया गया है:

• Gitcoin समस्या विवरण पृष्ठ पर और जानें
• चिप लगाना चाहते हैं? यहां अपना योगदान जोड़ें।
• प्रशन? चेकआउट गिटकोइन सहायता या गिटकोइन का विवाद
• $1,072,039.97 अधिक वित्त पोषित OSS कार्य Gitcoin इश्यू एक्सप्लोरर पर उपलब्ध है