Openwisp-utils: [qa] Pemeriksaan pesan komit gagal jika ada titik setelah masalah

Dibuat pada 7 Mei 2021  ·  9Komentar  ·  Sumber: openwisp/openwisp-utils

Semua 9 komentar

Bisakah saya melakukan masalah ini?

Yakin @keshavgbpecdelhi silakan

@devkapilbansal dapatkah Anda memberi tahu kami untuk melakukan perubahan (kode mana yang memerlukan modifikasi)

Saya pikir Anda perlu memperbaiki fungsi ini
https://github.com/openwisp/openwisp-utils/blob/1f73cc3d522478cd34bd6f10d3dfa98be12e4263/openwisp_utils/qa.py#L83

Saya mencoba melihat masalah itu dan menemukan bahwa regex mengembalikan #344. dari deskripsi panjang yang kemudian dicocokkan dengan #344 dari deskripsi singkat dan tes gagal karena #344. tidak sama dengan #344

Apakah kita harus menambahkan "." dalam kasus lewati?
skip_cases = [
# penggabungan
r'^Gabungkan permintaan tarik #[A-Za-z0-9.]* dari',
r"^Gabungkan cabang '(. ?)' ke",# rilisr'^[A-Za-z0-9.] rilis$',
r"^VERSI Berbenturan ke (. ?)$",r"^Bump (. ?) dari (. ?) ke (. ?)$",
]

Ini hanya untuk deskripsi singkat

@keshavgbpecdelhi apakah Anda mencoba melalui kode sekali dan mengerti apa yang terjadi di sana? Cara terbaik untuk memperbaiki kesalahan adalah dengan mencoba mereproduksinya secara lokal, melihat bagian kode mana yang bertanggung jawab untuk itu, menulis kasus uji yang pasti akan gagal kecuali kesalahan diperbaiki dan kemudian menyelesaikan kesalahan.

@keshavgbpecdelhi langkah pertama adalah menulis tes yang gagal untuk mereplikasi masalah, lihat https://github.com/openwisp/openwisp-utils/blob/master/tests/test_project/tests/test_qa.py#L79 -L135.

Apakah halaman ini membantu?
0 / 5 - 0 peringkat

Masalah terkait

nemesisdesign picture nemesisdesign  ·  5Komentar

atb00ker picture atb00ker  ·  13Komentar

atb00ker picture atb00ker  ·  9Komentar

gpetukhov picture gpetukhov  ·  3Komentar

jakoch picture jakoch  ·  3Komentar