Make getCompletionEntryDetails and getCompletionEntrySymbol parameters non-optional

[ghost]

Fixes #19455 (comment)

[ghost]

@mhegazy Let's get this in for the 2.6 release.

[mhegazy]

Following #19455 (comment), we should also change the API version

[mhegazy]

we also need that in release-2.6

[mhegazy]

@Andy-MS we need to port this to release-2.6 and add a note about it in https://github.com/Microsoft/TypeScript/wiki/API-Breaking-Changes

[angelozerr]

Thanks @mhegazy to have pinged me about this change. Is it possible to describe which methods change their signature at https://github.com/Microsoft/TypeScript/wiki/API-Breaking-Changes (it seems that it's just for completion)

In https://github.com/angelozerr/tslint-language-service we override only getSemanticDiagnostics and getCodeFixesAtPosition, so perhaps we need to do nothing. Thanks for your feedback!

[ghost]

@angelozerr Added the method names. If your code compiles that should mean that you don't need to change anything.

[angelozerr]

Thanks a lot @Andy-MS !