Telliot: استبدل كل `fmt.Errorf` بـ` الأخطاء.التفاف` و `الأخطاء. Errorf`

تم إنشاؤها على ٢٨ أكتوبر ٢٠٢٠  ·  15تعليقات  ·  مصدر: tellor-io/telliot

good first issue help wanted medium clean up

ال 15 كومينتر

حالة المشكلة: 1. فتح 2. بدأ 3. تم الإرسال 4. تم


__هذا الإصدار يحتوي الآن على تمويل قدره 10.0779 ليرة تركية (281.17 دولارًا أمريكيًا @ 27.9 دولارًا أمريكيًا / TRB) مرفق به .__

تضمين التغريدة
أنا مهتم بالعمل على هذه المشكلة ، قم بدعوتي من خلال Gitcoin

خطة عملي هي إجراء جميع التغييرات

developerfred ، تم

themandalore شكرا ، رائع!

يبدو أنه لا يزال هناك بعض fmt.Errorf في httpRetriever.go و logConfig.go (حزمة الاستخدام)
تضمين التغريدة

themandalore سأنظر هنا ، شكرًا

تضمين التغريدة

تبدو جيدة ، اسمحوا لي أن أعرف الخطوات التالية فيما يتعلق بـ gitcoin

themandalore يمكنك

مستخدم gitcoin الخاص بي: developerfred

developerfred ، لا يمكنني فعل أي شيء حقًا. سأتواصل معهم ، ولكن إذا استغرق الأمر أكثر من يوم أو يومين ، فما عليك سوى التواصل مرة أخرى وسنغلقه ونصلحه يدويًا

themandalore مثالي ، أنا أتحدث معهم أيضًا.
سيقومون بإصلاحه الآن ، ثم تكون العملية فقط للموافقة علي ثم الدفع.

themandalore انتهى ، الآن أنت بحاجة إلى موافقي على 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 ويصعب متابعته في العلاقات العامة لذا أفضل استخدام طريقة الالتفاف.

في المتصل ، يمكننا القيام بشيء مثل 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 بطلب لبدء العمل _ (الممولين فقط: الموافقة على العامل | رفض العامل ) _.

أنا على دراية بـ go ، وأود أن أقوم بهذه المهمة
4) تطبيق الايمان لبدء العمل _ (الممولين فقط: الموافقة على العامل | رفض العامل ) _.

سأكون قادرًا على استبدالها جميعًا يدويًا أو تلقائيًا.

تعرف على المزيد

حالة المشكلة: 1. فتح 2. بدأ 3. تم الإرسال 4. تم


__العمل مقابل 10.0779 TRB (283.87 USD @ $ 28.16 / TRB) تم تقديمه بواسطة _:


هل كانت هذه الصفحة مفيدة؟
0 / 5 - 0 التقييمات