Telliot: Ganti semua `fmt.Errorf` dengan `errors.Wrap` dan `errors.Errorf`

Status Masalah: 1. Buka 2. Mulai 3. Dikirim 4. Selesai

__Masalah ini sekarang memiliki dana sebesar 10,0779 TRB (281,17 USD @ $27,9/TRB) yang menyertainya.__

• Jika Anda ingin mengatasi masalah ini, Anda dapat 'mulai bekerja'
  
  
  
  • Ingin ikut campur? Tambahkan kontribusi Anda sendiri di sini .
  
  
  • Pertanyaan? Lihat Bantuan Gitcoin atau Obrolan Gitcoin
  
  
  • $1.052.111.78 lebih banyak Pekerjaan OSS yang didanai tersedia di Penjelajah Masalah Gitcoin
  
  

@themandalore
Saya tertarik untuk mengerjakan masalah ini, undang saya melalui Gitcoin

rencana kerja saya adalah membuat semua perubahan

@developerfred , dikirim. Anda adalah monyet luar angkasa kami sehubungan dengan menguji seluruh gitcoin ini, jadi beri tahu saya jika Anda memerlukan sesuatu

@themandalore Terima kasih, sempurna!

Sepertinya masih ada beberapa fmt.Errorf di httpRetriever.go dan logConfig.go (paket util)
@developerfred

@themandalore saya akan periksa di sini, terima kasih telah memberi tahu saya.

@themandalore selesai.

Terlihat bagus, beri tahu saya langkah selanjutnya terkait gitcoin

@themandalore Anda dapat menambahkan saya ke hadiah, ketika saya mencoba untuk menyatakan minat saya mendapatkan kesalahan. Saya melaporkan ke tim dan saya menganalisis kode Gitcoin untuk mengetahui alasannya.

pengguna gitcoin saya: @developerfred

@developerfred , saya tidak bisa melakukan apa-apa. Saya akan menghubungi mereka, tetapi jika butuh lebih dari satu atau dua hari, hubungi kembali dan kami akan menutupnya dan memperbaikinya secara manual

@themandalore sempurna, saya juga berbicara dengan mereka.
Mereka akan memperbaikinya sekarang, kemudian prosesnya hanya untuk menyetujui saya dan kemudian membayar.

@themandalore selesai, sekarang Anda harus menyetujui saya di Gitcoin, agar saya dapat mengirimkan PR.

Pembukaan kembali karena saya memiliki sesuatu yang lain dalam pikiran.
Kita perlu menggunakan errors.Wrap , errors.Wrapf dan juga menghapus kata-kata gagap untuk meningkatkan pesan log. Kata-kata seperti - can't, error, failed harus dihapus karena ini akan diulang dalam pesan log terakhir.

Berikut adalah beberapa contoh perubahan yang diperlukan:

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)

Catatan menghapus kata kesalahan - kami tidak membutuhkannya karena ini jelas dan pesan cetak akhir akan menyertakannya dua kali.

Menggunakan errors.Wrap memungkinkan kita untuk melakukan pencocokan kesalahan pada pemanggil dan umumnya lebih terstruktur:
lihat: https://blog.golang.org/go1.13-errors , Disebutkan penggunaan %w, tapi saya bukan penggemar karena cukup rawan kesalahan. - mudah digunakan %v dan sulit diikuti dalam PR jadi saya lebih suka menggunakan metode bungkus.

Di pemanggil, kita dapat melakukan sesuatu seperti if err == ErrNotFound (kesalahan pencocokan berdasarkan jenis) yang lebih sulit tanpa menggunakan metode bungkus.

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

harap diperhatikan lagi penghapusan kata failed itu tidak diperlukan. Juga di sini kami tidak menggunakan Wrapf , tetapi hanya Wrap

Status Masalah: 1. Buka 2. Mulai 3. Dikirim 4. Selesai

__Pekerjaan telah dimulai__.

Para pengguna ini masing-masing mengklaim bahwa mereka dapat menyelesaikan pekerjaan itu dalam 266 tahun dari sekarang.
Silakan tinjau rencana aksi mereka di bawah ini:

1) developerfred telah disetujui untuk mulai bekerja.

Saya ingin sekali mengerjakan masalah ini, rencana kerja saya adalah membuat semua penggantian
2) janus telah melamar untuk mulai bekerja _(Hanya penyandang dana: menyetujui pekerja | menolak pekerja )_.

Saya telah mengerjakan proyek Go sebelumnya dan saya juga memiliki keterampilan linux yang baik untuk menemukan dan mencocokkan kata. Saya hanya membutuhkan waktu berjam-jam untuk menyelesaikan tugas ini.
3) zyfrank telah melamar untuk mulai bekerja _(Hanya menyetujui pekerja | menolak pekerja )_.

Saya terbiasa dengan go, saya ingin mengambil tugas ini
4) iman telah diterapkan untuk mulai bekerja _(Hanya penyandang dana: menyetujui pekerja | menolak pekerja )_.

Saya akan dapat mengganti semuanya baik secara manual atau otomatis.

Pelajari lebih lanjut di halaman Detail Masalah Gitcoin .

Status Masalah: 1. Buka 2. Mulai 3. Dikirim 4. Selesai

__Karya untuk 10,0779 TRB (283,87 USD @ $28,16/TRB) telah dikirimkan oleh__:

• Pelajari lebih lanjut di halaman Detail Masalah Gitcoin
• Ingin ikut campur? Tambahkan kontribusi Anda sendiri di sini .
• Pertanyaan? Lihat Bantuan Gitcoin atau Obrolan Gitcoin
• $1,125.841,61 lebih banyak Pekerjaan OSS yang didanai tersedia di Penjelajah Edisi Gitcoin