Proposal: change the signature of the Constructor<T> type for use in mixins

[jolleekin]

Issue

Please change the signature of the Constructor<T> type (declared in decorators.ts) to use any instead of unknown so it can be used to create mixins. Without this change, anyone using lit-element will have to declare his/her own Constructor<T> type to create mixins.

export type Constructor<T> = {
  new (...args: unknown[]): T
};

// should be changed to

export type Constructor<T> = {
  new (...args: any[]): T
};

// or

export type Constructor<T> = new (...args: any[]) => T;

Versions

• lit-element: v0.6.1

[aadamsx]

If you know what needs to be done (it sounds like you do), submit a PR! :)

[justinfagnani]

This seems fine, but we should file an upstream issue on TypeScript

[jolleekin]

Seems like this is the upstream issue 14126