Skip to content
This repository has been archived by the owner on Nov 13, 2022. It is now read-only.

v3: fix create generics #118

Open
wants to merge 2 commits into
base: v3
Choose a base branch
from
Open

v3: fix create generics #118

wants to merge 2 commits into from

Conversation

viztea
Copy link
Collaborator

@viztea viztea commented Jun 18, 2021

No description provided.

* Creates a new player manager with the supplied options.
*
* @param options The options to supply the player manager.
* @param klass The provider class to instantiate.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would suggest target is better word here than klass

* @param options The options to supply the player manager.
* @param klass The provider class to instantiate.
*/
export function create<M extends Manager<any, any>>(options: ExtractOptions<M>, klass?: Class<M>): M
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same as above klass => target

*/
export function create<M extends Manager<any, any>>(options: ExtractOptions<M>, klass?: Class<M>): M

export function create<M extends Manager<any, any>>(options?: ExtractOptions<M>, klass?: Class<M>): M {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same as above klass => target

@anishshobithps anishshobithps added documentation Improvements or additions to documentation enhancement New feature or request labels Jun 18, 2021
@anishshobithps anishshobithps added this to the v3 milestone Jun 18, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants