FindInFiles converted to command and accepting arguments

[jwikman]

Tries to solve #29405

[msftclas]

All CLA requirements met.

[jwikman]

@roblourens - Code is modified to use an args variable when calling setSearchParameters (which is renamed)

Did you get a few minutes to think about the trigger search parameter?

[roblourens]

Merging this, but I am going to keep thinking about triggerSearch, and I reserve the right to revert it before the next stable release :)

Thanks for the PR!

[roblourens]

Btw, thinking about it more, I think all of this is fine, including allowing an extension to trigger the search, and I look forward to seeing what people use it for!

[jwikman]

Thanks @roblourens, that was good news on a Friday night. 👍

[jwikman]

@roblourens , is this included in the v1.34 release? Didn't see anything in the release notes. ;-)

[roblourens]

Yep it's there

[jwikman]

Yep it's there

OK, thanks

[roblourens]

Oh I forgot to add this to the milestone so it wasn't included in the release notes. I'll fix that and it will show up next time the website is updated. Sorry!