-
Notifications
You must be signed in to change notification settings - Fork 12.9k
Closed
Labels
BugA bug in TypeScriptA bug in TypeScriptFix AvailableA PR has been opened for this issueA PR has been opened for this issue
Milestone
Description
Bug Report
π Search Terms
class async override
π Version & Regression Information
- This changed between versions 4.2.0 and nightly
β― Playground Link
Playground link with relevant code
π» Code
class Base {
method1() {}
method2() {}
}
class Test extends Base {
public override async method1() {
return Promise.resolve(1);
}
public async override method2() {
return Promise.resolve(1);
}
}
π Actual behavior
No errors
π Expected behavior
Possibly error on either override async
or async override
.
My understanding (from #43533) is that you prefer requiring a specific modifier order. If that's the case, I would have expected an error here.
nguyenlamllltonivj5, Kingwl and Austaras
Metadata
Metadata
Assignees
Labels
BugA bug in TypeScriptA bug in TypeScriptFix AvailableA PR has been opened for this issueA PR has been opened for this issue