fix: Using container.get/set methods with abstract class id

[dilame]

Description

Here is the error message

TS2769: No overload matches this call.
  The last overload gave the following error.
    Argument of type 'typeof IgpapiHttp' is not assignable to parameter of type 'ServiceIdentifier<unknown>'.
      Type 'typeof IgpapiHttp' is not assignable to type 'Constructable<unknown>'.
        Cannot assign an abstract constructor type to a non-abstract constructor type.
src/igpapi.ts (56:24)

56     this.container.set(IgpapiHttp, http);
                          ~~~~~~~~~~

  ../../node_modules/typedi/types/container-instance.class.d.ts:50:5
    50     set<T = unknown>(token: ServiceIdentifier, instance: T): this;
           ~~~
    The last overload is declared here.

Here is the same error screenshot (maybe it's more comfortable to read the colored text for someone)

Minimal code-snippet showcasing the problem

abstract class Foo {}

const container = new ContainerInstance('');

container.set(Foo, 1)

It doesn't work for TypeScript "strict": true mode,

[NoNameProvided]

Hmm, this should work. I will need to take a look. Thanks for the report.

[NoNameProvided]

Fixed in https://github.com/typestack/typedi/commit/aff2542f7e81383d1c9613109fdef2825b5c7619. It will be included in the next release.

[dilame]

Thank you! When are you planning the next release?

[finwo]

@NoNameProvided Is there any update on the next release?

[NoNameProvided]

Hi! I am focusing on class-validator currently, but I will try to pick up the reins here as well and see what will be released and make it happen.

[viict]

@NoNameProvided hey, any update on this release? I just hit the same issue

[finwo]

I eventually abandoned the library in favor of writing my own.