microsoft / microsoft/TypeScript
Expose isTypeReference function in public API
まだ誰も着手していません。
- 主要言語
- Go
- スター
- 111k
- フォーク
- 14.3k
- 平均マージ
- 2日 4時間
- マージ済み PR(30日)
- 132
説明
Search Terms
isTypeReference
Suggestion
The public compiler API exposes a getTypeArguments method on the type checker:
But provides no way to check if a type is a TypeReference. Users of the TS compiler API have to roll their own, which is error-prone. I suggest that if TypeReference is exposed in the public API, there should be a way to assert whether a type is a TypeReference.
Use Cases
See above
Examples
Currently, public uses of getTypeArguments must do something like this to assert that a type is a TypeReference:
function isTypeReference(ty: ts.Type): ty is ts.TypeReference {
return 'typeArguments' in ty;
}
const ty = getType();
if (isTypeReference(ty)) {
getChecker().getTypeArguments(ty);
}
With an exposed type guards, this becomes
const ty = getType();
if (ts.isTypeReference(ty)) {
getChecker().getTypeArguments(ty);
}
Checklist
My suggestion meets these guidelines:
- This wouldn't be a breaking change in existing TypeScript/JavaScript code
- This wouldn't change the runtime behavior of existing JavaScript code
- This could be implemented without emitting different JS based on the types of the expressions
- This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
- This feature would agree with the rest of TypeScript's Design Goals.
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
まず、公開されている getTypeArguments メソッドとパブリックな TypeReference API の周辺にある lib/typescript.d.ts を調べます。完成した API が ts.isTypeReference を公開していること、および例でそれを使って getTypeArguments の呼び出しをガードできることを確認します。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- typescript
- 領域
- compilers
- issue の種類
- 機能追加
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 活発さ
- 停滞
- 明瞭さ
- 明確に書かれている
- 初心者へのやさしさ
- 45/100