Make WorkspaceReferencesParams.query partial

[felixfbecker]

No description provided.

[slimsag]

Where can I read about what this does? I Googled a little bit but only came up with mixins etc

[beyang]

@slimsag see discussion here: microsoft/TypeScript#4889.

I think this is not strictly necessary to make things typesafe, but it does provide a hint to the consumer of the spec that not all the fields have to be present in the query.

[felixfbecker]

It makes all properties optional.
Docs here: https://www.typescriptlang.org/docs/handbook/advanced-types.html#mapped-types
In my case it is pretty important to know whether all fields I define (which includes a unqiue symbol ID) will be present or not, so I thought it would be good to clear it up in the spec

[slimsag]

LGTM

[slimsag]

Thanks for the links, BTW!