Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

autocomplete of params/var replaces needed code #127

Closed
bmoore-msft opened this issue Aug 30, 2018 · 3 comments
Closed

autocomplete of params/var replaces needed code #127

bmoore-msft opened this issue Aug 30, 2018 · 3 comments

Comments

@bmoore-msft
Copy link
Contributor

When using autocomplete for a param or variable name in the middle of an expression, too much code is removed upon completion.

Take the following expression:

"[if('true', variables('one'), variables('two'))]"

Now replace true with:

parameters('

this will bring up the autocomplete list, TAB select something from the list. At this point everything up to the next single quote is replaced and you end up with:

[if(parameters('enableBootDiags')one'), variables('two'))]

If autocomplete stopped replacement at the comma or whitespace this would work perfectly.

@StephenWeatherford
Copy link
Contributor

@bmoore-msft This appears to have been fixed with #551. Can you verify this and the new intellisense experiences (parameter files and resourceId) in https://dev.azure.com/ms-azuretools/AzCode/_build/results?buildId=18384&view=results? Thx.

@bmoore-msft
Copy link
Contributor Author

Looks like it - very cool...

@StephenWeatherford
Copy link
Contributor

Shipped yesterday

@vscodebot vscodebot bot locked and limited conversation to collaborators Jun 11, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants