You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I'll explain with a possible use case:
Get-SNOWObject is used as a template function for all other Get-SNOW* (table) commands.
It's parameters are pulled through to form the framework of those other functions (into the dynamic param block)
Default values are not pulled through as it's not part of the 'Get-Command' dataset.
Now lets say that by default, we wanted all Get-SNOW* functions to default the -DisplayValue to 'true'.
I could set this in the Get-SNOWObject command, but it would not apply to all others templating from this.
This might also be a better method for parsing the script block pushed into Invoke-SNOWBatch, rather than using the current regex method. It's something to explore anyway.
insomniacc
added
the
scoping
A potential idea that needs to be scoped out first to understand if it's feasible
label
Mar 12, 2023
insomniacc
removed
the
scoping
A potential idea that needs to be scoped out first to understand if it's feasible
label
Mar 23, 2023
I'm not sure if there's an alternative on how to do this in the way that I'm hoping for, but for now I don't have a specific requirement that needs this, it would just be a nice to have. Going to stick it on the back burner until either something new comes up or I decide to shelve it.
I'll explain with a possible use case:
Get-SNOWObject is used as a template function for all other Get-SNOW* (table) commands.
It's parameters are pulled through to form the framework of those other functions (into the dynamic param block)
Default values are not pulled through as it's not part of the 'Get-Command' dataset.
Now lets say that by default, we wanted all Get-SNOW* functions to default the -DisplayValue to 'true'.
I could set this in the Get-SNOWObject command, but it would not apply to all others templating from this.
Possible Solution:
Ast parsing
This is not currently presenting an issue, although might be a nice to have for the use case specified above.
The text was updated successfully, but these errors were encountered: