microsoft / microsoft/TypeScript
`deleteCount` param to `splice()` is not optional in es5 types
オープン
まだ誰も着手していません。
Bug
Domain: lib.d.ts
- 主要言語
- Go
- スター
- 111k
- フォーク
- 14.3k
- 平均マージ
- 2日 4時間
- マージ済み PR（30日）
- 132
説明
TypeScript Version: 3.5.1
Search Terms: splice deleteCount
Code
Set the target to ES5 then
[1, 2, 3].splice(0); // should error. deleteCount missing
[1, 2, 3].splice(0, 3); // ok
Expected behavior:
[1, 2, 3].splice(0); not allowed
Actual behavior:
[1, 2, 3].splice(0); allowed
Playground Link: this
deleteCount only become optional in ES6
ES5: https://www.ecma-international.org/ecma-262/5.1/#sec-15.4.4.12
ES6: https://www.ecma-international.org/ecma-262/6.0/#sec-array.prototype.splice
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
まず、target を ES5 に設定して TypeScript Playground で例を再現し、次に Array.splice の ES5 と ES6 の宣言を比較します。ES5 では deleteCount の省略を拒否し、ES6 では引き続き許可すれば、修正は完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- typescript
- 領域
- compilers
- issue の種類
- バグ
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 活発さ
- 停滞
- 明瞭さ
- 明確に書かれている
- 初心者へのやさしさ
- 45/100