1.9.0 では、 _.first([], 10)
はundefined
代わりに[]
undefined
返します。
これは大きな問題だと思います :stuck_out_tongue:
うわ…これは意図した変更だったように見える… https://github.com/jashkenas/underscore/pull/2513
個人的には、 _.first([1,2,3], 2)
が[1,2]
ことから、 _.first([], 2)
が[]
と期待すると思います。
明示的に指定されたアイテムの数で呼び出された場合、それは私には厄介な小さな回帰のように見えます。 他の意見は? @jridgewell?
そうは言っても、 _.first([], 1)
はどうあるべきかという疑問が生じます。
おっと、私の側のレビューが悪い。 n
引数を渡すと、常に配列が返されます。
心配いりません。1.9.1 は、発生した他のクリーンアップで実行できます — 加えて、他の未解決の PR のいくつかを解決するかもしれません...
グラシアス。 これは今日も私たちを噛んだ。
これは、私の場合、より大きなアプリのさまざまな場所で、エッジケースで非常にあいまいなバグにつながります。
優先すべきではないでしょうか。
これも私たちを噛んだ。 幸いなことに、実際に公開する前に、いくつかの単体テストでキャッチされました。 早急に直していただけるとありがたいです。
よし! Underscore 1.9.1 が公開され、このバグ修正が含まれています。
ありがとう@jashkenas
最も参考になるコメント
よし! Underscore 1.9.1 が公開され、このバグ修正が含まれています。