Add support to pass includeDeclaration as True for textDocument/references

[EmasXP]

When searching for references, I would like to include the declarations in the result.

My thinking is that this could be done by adding a include_declaration argument to lsp_symbol_references. The default value should in that case be False.

What do you think?

[jwortmann]

Sounds like a good idea to me. It seems that the False value has always been hardcoded for this, probably because it's inline with Sublime's built-in "Goto References..." behavior. The new command argument would mean that you need to add your own context menu entry and/or key binding if you want to change this behavior.

An alternative would be to expose this as a new global setting, which would be more user friendly. But new settings mean more maintenance work in the future, and I think that the impact of changing this option would be rather small. So I would say your suggestion is better.

[EmasXP]

I can do a PR for this if you want? Disclosure: that would be my first ever PR. I am happy to do it though 🙂

[predragnikolic]

@EmasXP feel free to open a PR :)