Open kayahr opened 4 years ago
I think (having recently looked at it) @extends
only supports classes declared with class
not old-style es5 classes.
I'm currently evaluating if TypeScript can be used to type-check old closure compiler projects which use JSDoc annotations for typing and to generate *.d.ts files for it so the old projects can be easily used within new typescript projects.
I have this exact use case. Anyone done any digging as to how difficult this would be to support? I have no experience with TypeScript internals.
@jpoehnelt Full support is not hard, but would be tricky since it messes with the core class/this
instantiation code. The code would be fairly similar to support for @class
; create a getEffectiveExtendsClause
that checks extends
AND @extends
, then make sure that it's used everywhere. There are likely quite a few places that need to be updated.
I have this use case as well - I have a constructor function Foo (that can't be turned into class
syntax) that has util.inherits(Foo, EventEmitter);
, and I have no way to tell TS that the constructor extends EventEmitter, with jsdoc or a d.ts.
+1 please fix this
TypeScript Version: 3.7.5 and 3.8.0-dev.20200123
Search Terms: extends not attached to a class
I'm currently evaluating if TypeScript can be used to type-check old closure compiler projects which use JSDoc annotations for typing and to generate *.d.ts files for it so the old projects can be easily used within new typescript projects. But unfortunately the
@extends
annotation is not working as expected. I would understand if TypeScript simply didn't support it (Then this would be a feature request) but the error message error TS8022: JSDoc '@extends' is not attached to a class thrown by the compiler suggests that there is some support for but it doesn't work as expected.Please note that the real class inheritance code which actually extends class
Sub
fromBase
is omitted in the code example below because it is irrelevant to the compiler.Code
Expected behavior:
Typescript should recognize type
Sub
to be a class which extends classBase
.Actual behavior:
Compilation fails with this error: