Mopidy: Pernyataan tegas digunakan di luar tes

Saya menandai ini sebagai masalah pertama yang bagus. Solusi langsung adalah mengubah dari:

assert condition, "Error message"

Ke:

if not condition:
    raise AssertionError("error message")

Ini membuat kode berperilaku dengan cara yang sama persis seperti hari ini, tetapi tidak akan dioptimalkan saat menjalankan Python dengan -O .

@jodal , haruskah saya menyelesaikan ini atau membiarkan kontributor pertama kali lainnya menyelesaikannya?

Apalagi jika Anda ingin fresher menyelesaikannya maka saya butuh saran dari Anda yaitu bagaimana jika masalah ini kita pecah menjadi beberapa masalah lain? Maksud saya memecahkan masalah menurut file.

Jangan ragu untuk menangani ini jika Anda tertarik!

Jika sangat banyak contoh masalah, masuk akal untuk membaginya menjadi beberapa permintaan tarik berdasarkan area basis kode. Namun, sepertinya tidak banyak kasus, jadi saya sarankan membuat satu PR saja untuk memperbaiki semua kasus.

Jumlah assert per file, tidak termasuk tes:

$ rg -c assert mopidy
mopidy/ext.py:3
mopidy/core/actor.py:1
mopidy/core/tracklist.py:8
mopidy/audio/utils.py:1
mopidy/core/playback.py:1

Hai @pnijhara @jodal Saya ingin mengerjakan masalah ini. Bisakah Anda memberi saya ini sebagai kontributor pertama kali.

Ada informasi di https://docs.mopidy.com/en/latest/contributing/ dan juga di https://docs.mopidy.com/en/latest/devenv/ tentang cara menyiapkan lingkungan pengembangan dan menjalankan pengujian dan linter dll.