Added optional mpv_args
argument to types
#102
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes an issue where typescript will complain about providing the mpv_args argument on the Node_MPV.start() method
To give you some context, projects that were not written in TypeScript can still provide an index.d.ts file. This way, TypeScript can interpret the expected input and output for every function you expose in node-mpv.
However, there was a mistake in the type definition for the Node_MPV.start method.
Take a look at this snippet and the typescript error it causes:

It says 0 arguments are expected in this function, but clearly in _StartStop.js, you defined an optional array argument
mpv_args
:This
mpv_args
is missing from the index.d.ts file on L635:Hence, this PR. I changed only the type definition:
Array<string|number>
Hope this is clear, and close enough to what your function actually expects.