-
Notifications
You must be signed in to change notification settings - Fork 27
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
The -Scope parameter for Export-Configuration has a parameterset (and it should not) #8
Comments
If you remove the ManualOverride ParameterSet from Scope, all three of these Scenarios should pass. Even if you have the Company set to 'Unknown', you can still specify the AuthorName and it will successfully update the Config. |
I would love to PR this, but I see some development is happening off the master branch. I see that Jaykul is actively working on the build process, and I'm getting nowhere with the build scripts as they exist in master. Should I hold off? @Jaykul is it your intention to merge / deploy to the gallery any time soon? |
LOL. Yeah. I have a couple more pieces to do in the build (I'm working on a Script Analyzer action), and then I was thinking of trying to auto-publish builds of master ... It's all a little mad. I made a new CI build in Github Actions because I was having weird problems with the Azure build (and I wanted to learn Actions anyway). It turned out it wasn't properly importing the new build output for some reason when it ran the tests. Fixed now, anyway. |
Ok, that stuff is done (enough). I still want to make some improvements, but the build is working and running tests and script analyzer, and it's merged to master. Thanks for your patience, @fsackur |
No, thank YOU. I'll pick this back up when my day job calms down a little. |
We need to be able to specify a value for
Export-Configuration -Scope
regardless of ParameterSet (just like you can forGet-StoragePath
).Currently, it has it's ParameterSet to the
ManualOverride
set, so you can only use it when specifying the -Name and -Company by hand (as in the first example below).I want all three of these to work:
The text was updated successfully, but these errors were encountered: