-
Notifications
You must be signed in to change notification settings - Fork 18
Update build and release pipeline #131
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
base: v2
Are you sure you want to change the base?
Conversation
6a6a562 to
51bf0f6
Compare
51bf0f6 to
c9bcc51
Compare
assets/powershell.config.json
Outdated
| "PSForEachObjectParallel", | ||
| "PSImplicitRemotingBatching", | ||
| "Microsoft.PowerShell.Utility.PSDebugRunspaceWithBreakpoints", | ||
| "PSDesiredStateConfiguration.InvokeDscResource" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was for v3 and should no longer be needed
.github/workflows/ci-test.yml
Outdated
|
|
||
| - name: Test Windows PowerShell | ||
| run: | | ||
| Install-Module Pester -Scope CurrentUser -Force -SkipPublisherCheck |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You should specify the version needed
| Function global:Test-IsInvokeDscResourceEnable { | ||
| # ExperimentalFeature is only available in PowerShell 7+ | ||
| if ($PSVersionTable.PSVersion.Major -lt 7) { | ||
| return $false | ||
| } | ||
| return [ExperimentalFeature]::IsEnabled("PSDesiredStateConfiguration.InvokeDscResource") | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was only for PSDSC v3
New build and release pipeline for a 2.0.8 release.