eugenhu

The `matches` variable in `ActionCallerTestCase.test_get_contents()` also looks unused after being redefined on line 2043: https://github.com/SCons/scons/blob/9861322b2306f17fc9a9037791fa689427c7275e/SCons/ActionTests.py#L2043-L2055 Should I remove it?

I believe the sticking point from last time was what `Value.get_text_contents()` should return: concatenation of all child contents (old behaviour, possibly large memory usage) or concatenation of child csigs (breaks...

This seems to be working now, not sure when exactly it was introduced but see > https://github.com/microsoft/python-language-server/issues/19

Haven't tried 'quick fix', but the auto import option is there for me when I do 'code actions' on the unimported module.