Httpie: Tambahkan opsi 'tenang' atau 'diam' untuk menggunakan httpie dalam skrip

Dibuat pada 3 Jun 2020  ·  12Komentar  ·  Sumber: httpie/httpie

Bendera sederhana untuk menghindari keluaran yang tidak perlu saat menggunakan skrip di dalam.

contributor-friendly feature

Komentar yang paling membantu

Ya saat ini sedang mengerjakannya :)

Semua 12 komentar

Rasional: Mengapa banyak perintah memberikan opsi "tenang"?

Kita dapat mengatasinya dengan mengganti env.stdout dan env.stderr dengan os.devnull pada saat yang tepat. Alih-alih menyentuh os.devnull secara langsung saat runtime, itu harus ditambahkan sebagai Environment.devnull seperti yang kita lakukan dengan aliran standar:

https://github.com/jakubroztocil/httpie/blob/492687b0dafb7bec0d6281d019bb5f4f60439346/httpie/context.py#L32 -L35

…dan sejak saat itu hanya referensi env.devnull . Ini akan memungkinkan untuk menulis tes yang baik. Jadi pada dasarnya hanya ada satu kondisional di suatu tempat dan semua perilaku yang ada harus tetap utuh.

Apa yang seharusnya menjadi bendera?

@paulolieuthier --quiet terdengar cukup standar

Hai! Apakah seseorang mengerjakan masalah ini atau dapatkah itu diambil?

@ nb2838 saya tidak. Jadi, kecuali @paulolieuthier melihat ini, itu harus diperebutkan.

Silakan @nb2838! saya tunggu rilisnya :)

@nb2838 Apakah Anda masih mengerjakan ini? Saya ingin mengangkat masalah ini jika tidak!

Ya saat ini sedang mengerjakannya :)

Saya telah mengerjakan masalah ini dan saya pikir saya memiliki solusi yang berhasil. Satu-satunya masalah saya sejauh ini adalah saya tidak tahu pada titik mana tepatnya untuk menutup file os.devnull. Jika saya tidak menutupnya dan membiarkan pekerjaan dilakukan oleh pengumpul sampah, kode saya berfungsi dengan baik. Namun, ketika saya mencoba menutupnya dengan menambahkan
finally: env.dev.null.close() di akhir main di core.py Saya mendapatkan kesalahan jika saya menggunakan opsi --download. Saya ingin tahu apakah Anda memiliki saran untuk menangani masalah ini.

@nb2838 tolong buka PR dan saya akan melihatnya

945 digabung dan ditutup.

Apakah halaman ini membantu?
0 / 5 - 0 peringkat