-
Notifications
You must be signed in to change notification settings - Fork 81
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
ARM template user defined functions are not supported #122
Comments
Talked with an Ignite attendee who has run into this as well, here's his blog about how he's using functions and an example arm template that we could use to test this. https://azurekid.blog/2018/06/17/arm-templates-and-azure-governance/ |
Awesome
You talked to Rogier Dijkman. He and I are colleagues and we both worked on
this already :)
It would be great to have VSCodesupport it as well
…On Thu, 27 Sep 2018 at 16:37, Matt Hernandez ***@***.***> wrote:
Talked with an Ignite attendee who has run into this as well, here's his
blog about how he's using functions and an example arm template that we
could use to test this.
https://azurekid.blog/2018/06/17/arm-templates-and-azure-governance/
https://github.com/azurekid/blog/blob/master/virtualMachine.json
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#122 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AFOwMy6uIXuvuHdLLX9qCHCuuePgLLiBks5ufOK8gaJpZM4VUsxg>
.
|
Thanks for the feedback. We still have to prioritize, so it's helpful to know how many people this would help. |
+1 for this fix, look forward to having this feature implemented! |
Good to know, thanks! |
+1; errors are also misleading as parameters consumed by the function are marked as missing as the extension confuses them template parameters |
+1 too... having an error reported each time we use a function from a local namespace actually hide real error(s) on the same line |
+1 |
Shamelessly promoting my blog about user defined functions. Hopefully it inspires more to use it. |
@azurekid :-) Their usage appears to be somewhat low right now, but I agree they ought to be used more. |
+1. I think the biggest "issue" is that it is marked as errors in the file.. |
Okay, thanks. This is pretty high on the list now. |
I considered whether there was a way to just get rid of the false errors in a quick manner, but I think it's too complicated for that - for instance, we user-defined functions, you can have parameters defined in a scope not at the top level, which we don't understand yet. About to start working on this, so hopefully in the next release (after the one about to ship). Thanks for your patience. |
In progress... |
Fix went out today with this fix, thanks. |
VSCode does not yet support ARM templates as definded here:
https://docs.microsoft.com/en-us/azure/azure-resource-manager/resource-group-authoring-templates#functions
Please could this be implemented?
The text was updated successfully, but these errors were encountered: