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
We have various action inputs that take a fixed set of values, or a list of a fixed set of values.
Some examples: type - must be either deployment or deploymentStacks scope - must be either tenant, managementGroup, subscription or resourceGroup
Currently our validation logic is strict - the casing has to match exactly. This means that we would reject scope: Tenant or type: deploymentstacks.
As raised in #85 (comment), this is a bit odd with our environment input, as it requires camel case (azureCloud, azureChinaCloud, azureGermanCloud, azureUSGovernment), whereas the environment input for the "azure-login" action (which this is commonly used with) requires Pascal Case (AzureCloud, AzureChinaCloud, AzureGermanCloud, AzureUSGovernment) - see https://github.com/marketplace/actions/azure-login#environment
IMO the best option for dealing with this inconsistency would be to relax our validation such that we don't care about casing for enum inputs.
The text was updated successfully, but these errors were encountered:
We have various action inputs that take a fixed set of values, or a list of a fixed set of values.
Some examples:
type
- must be eitherdeployment
ordeploymentStacks
scope
- must be eithertenant
,managementGroup
,subscription
orresourceGroup
Currently our validation logic is strict - the casing has to match exactly. This means that we would reject
scope: Tenant
ortype: deploymentstacks
.As raised in #85 (comment), this is a bit odd with our
environment
input, as it requires camel case (azureCloud
,azureChinaCloud
,azureGermanCloud
,azureUSGovernment
), whereas theenvironment
input for the "azure-login" action (which this is commonly used with) requires Pascal Case (AzureCloud
,AzureChinaCloud
,AzureGermanCloud
,AzureUSGovernment
) - see https://github.com/marketplace/actions/azure-login#environmentIMO the best option for dealing with this inconsistency would be to relax our validation such that we don't care about casing for enum inputs.
The text was updated successfully, but these errors were encountered: