Visual Basic support

[sharwell]

This pull request adds the Visual Basic support implemented by @AdamSpeight2008. Fixes #6.

[tugberkugurlu]

I guess file renames wiped all file history but there is nothing to do there 😞 looks good to me.

[sharwell]

Must be a GitHub limitation of anything. Git itself has no trouble tracking files through the rename here, even for operations like rebase.

[tugberkugurlu]

I cannot seem to get it working when I F5 to test it inside the experimental hive:

Am I doing something wrong?

[tugberkugurlu]

It seems that the problem is the node target. It only sees the node as ParameterSyntax when you select the whole parameter:

I would assume to get the refactoring suggestion when my cursor is around the parameter name as below but the node seems to be the ModifierIdentiferSyntax:

[tugberkugurlu]

do we have nameof operator in VB?

[sharwell]

I would suggest making this a follow-up issue.

[tugberkugurlu]

👍

[sharwell]

Is there anything here that still needs to be fixed prior to the initial "merge"? Since the request contains file moves it helps to include it early so people can work from a new reference point.

[tugberkugurlu]

@sharwell Can we do something for this (I have no idea if it's safe to watch for ModifierIdentiferSyntax node instead of ParameterSyntax)? #15 (comment)

It seems pretty annoying but if this is the way it should be,

[sharwell]

If the behavior is unintuitive we can file a separate issue.

[tugberkugurlu]

That's fine, I'm getting it in then. I will open up the issues as well.