microsoft / microsoft/TypeScript
redux-orm broken by #43624
オープン
@weswigham がすでに取り組んでいます。
2021年4月28日 から。
Bug
Domain: This-Typing
Rescheduled
- 主要言語
- Go
- スター
- 111k
- フォーク
- 14.3k
- 平均マージ
- 1日 19時間
- マージ済み PR(30日)
- 117
説明
The nightly dtslint run for 4/28 fails on redux-orm, with new errors on examples like:
export default class Model<MClass extends typeof AnyModel = typeof AnyModel, Fields extends ModelFieldMap = any> {
// ...
readonly ref: Ref<this>;
// ...
}
export class AnyModel extends Model {}
export type Ref<M extends AnyModel> = {
[K in keyof RefFields<M>]: ModelFields<M>[K] extends AnyModel ? IdType<ModelFields<M>[K]> : RefFields<M>[K];
};
ERROR: 118:23 expect TypeScript@4.3 compile error:
Type 'this' does not satisfy the constraint 'AnyModel'.
Type 'Model<MClass, Fields>' is not assignable to type 'Model<typeof AnyModel, any>'.
Type 'typeof AnyModel' is not assignable to type 'MClass'.
'typeof AnyModel' is assignable to the constraint of type 'MClass', but 'MClass' could be instantiated with a different subtype of constraint 'typeof AnyModel'.
Almost certainly a result of https://github.com/microsoft/TypeScript/pull/43624, but it could be #42449 or #43835, since they went in on the same day.
If this is an intended result of that PR, can you fix up redux-orm? A naive change to Ref<M extends Model> doesn't work.
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
評価
この issue はまだ評価されていません。