Skip to content

(MODULES-8355) Add pwsh provider #238

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

Merged

Conversation

glennsarti
Copy link
Contributor

@glennsarti glennsarti commented Dec 13, 2018

PowerShell 6 now uses a different binary and has different installation locations
compared to Windows PowerShell. This commit adds a new provider called pwsh
which will execute scripts but using PowerShell 6 (pwsh). Later commits will add
advanced features, but this commit adds a basic provider with tests that works
only on Windows, with PowerShell 6 in the default installation location.

@glennsarti glennsarti force-pushed the modules-8355-add-pwsh-provider branch from 7e9cb39 to 97aefc5 Compare December 17, 2018 01:51
PowerShell 6 now uses a different binary and has different installation locations
compared to Windows PowerShell.  This commit adds a new provider called pwsh
which will execute scripts but using PowerShell 6 (pwsh).  Later commits will add
advanced features, but this commit adds a basic provider with tests that works
only on Windows, with PowerShell 6 in the default installation location.
@glennsarti glennsarti force-pushed the modules-8355-add-pwsh-provider branch from 97aefc5 to 01aae9a Compare December 17, 2018 02:05
@glennsarti glennsarti changed the title {WIP} (MODULES-8355) Add pwsh provider (MODULES-8355) Add pwsh provider Dec 17, 2018
@glennsarti
Copy link
Contributor Author

glennsarti commented Dec 18, 2018

Note I have run beaker with a powershell6 role on Windows and they pass successfully.

@michaeltlombardi michaeltlombardi merged commit 9f75de3 into puppetlabs:master Dec 18, 2018
@glennsarti glennsarti deleted the modules-8355-add-pwsh-provider branch January 9, 2019 01:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants