func getKeyNameFromDeploymentAndSubnet(subnetType SubnetType, deploymentType DeploymentType, keyNameMap map[SubnetType]string) string {
if (deploymentType == NoDeplo
Since you have a return statement in your first if
, you may just drop the else
statement. The second if
is not going to be reached anyway if the first ones conditions are met.
func getKeyNameFromDeploymentAndSubnet(subnetType SubnetType, deploymentType DeploymentType, keyNameMap map[SubnetType]string) string {
if (deploymentType == NoDeployment || deploymentType == PDBAWindows || deploymentType == AgentDeployment) {
return keyNameMap[subnetType]
}
if (deploymentType == AnsibleDeployment) {
return "bar"
}
return "foo"
}