-
Notifications
You must be signed in to change notification settings - Fork 4
Open
Labels
bugThe issue is a bug.The issue is a bug.good first issueThe issue should be easier to fix and can be taken up by a beginner to learn to contribute on GitHubThe issue should be easier to fix and can be taken up by a beginner to learn to contribute on GitHubhelp wantedThe issue is up for grabs for anyone in the community.The issue is up for grabs for anyone in the community.
Description
Should this line be elseif (-not [System.String]::IsNullOrEmpty($BaseDirectory)) to solve that the throw statement is never reached?
| elseif ($null -ne $BaseDirectory) |
It looks like the line should be changed to what is suggested above, but that will fail the test, and looking at this it need to be looked at further to see if it should be refactored in some way depending on how the function is used from the classes (how ImportLocalization() is used).
Metadata
Metadata
Assignees
Labels
bugThe issue is a bug.The issue is a bug.good first issueThe issue should be easier to fix and can be taken up by a beginner to learn to contribute on GitHubThe issue should be easier to fix and can be taken up by a beginner to learn to contribute on GitHubhelp wantedThe issue is up for grabs for anyone in the community.The issue is up for grabs for anyone in the community.