Dengan 1.9.0 _.first([], 10)
mengembalikan undefined
bukan []
.
Saya pikir ini adalah masalah besar :stuck_out_tongue:
Wow... Sepertinya ini adalah perubahan yang disengaja ...
Secara pribadi, saya pikir Anda akan mengharapkan _.first([], 2)
menjadi []
mengikuti dari _.first([1,2,3], 2)
menjadi [1,2]
Itu memang terlihat seperti kemunduran kecil yang buruk bagi saya — ketika dipanggil dengan jumlah item yang ditentukan secara eksplisit. Ada pendapat lain? @jridgell?
Yang mengatakan, itu menimbulkan pertanyaan tentang apa _.first([], 1)
seharusnya.
Ups, ulasan buruk di pihak saya. Melewati argumen n
harus selalu mengembalikan array.
Jangan khawatir, kami dapat melakukan 1.9.1 dengan pembersihan lain yang muncul — ditambah mungkin menyelesaikan beberapa PR terbuka lainnya...
terima kasih. yang ini juga menggigit kita hari ini.
Yang ini mengarah ke bug yang sangat tidak jelas dalam kasus tepi di banyak berbagai tempat di aplikasi yang lebih besar dalam kasus saya.
Bukankah itu harus menjadi prioritas?
Yang ini juga menggigit kita. Untungnya itu tertangkap oleh beberapa unit test sebelum kami meluncurkannya secara langsung. Saya akan berterima kasih untuk perbaikan cepat.
Baik! Garis bawah 1.9.1 sekarang diterbitkan, dengan perbaikan bug ini di dalamnya.
Terima kasih @jashkenas
Komentar yang paling membantu
Baik! Garis bawah 1.9.1 sekarang diterbitkan, dengan perbaikan bug ini di dalamnya.