Skip to content

Move Docker tests to GitHub Actions - #71

Merged
mlocati merged 8 commits into
masterfrom
docker-tests-in-github-actions
Oct 28, 2026
Merged

mlocati merged 8 commits into
masterfrom
docker-tests-in-github-actions

Conversation

@mlocati

@mlocati mlocati commented Oct 31, 2020

Copy link
Copy Markdown
Owner

No description provided.

@mlocati
mlocati force-pushed the docker-tests-in-github-actions branch 8 times, most recently from b2ebdc0 to 548f928 Compare October 31, 2020 19:25
@mlocati
mlocati force-pushed the docker-tests-in-github-actions branch 2 times, most recently from a078f14 to b8fc2db Compare April 19, 2021 19:18
@mlocati

mlocati commented Oct 20, 2022

Copy link
Copy Markdown
Owner Author

Test

?

Only the last patch release of PHP 7.x is still available for download.
Use ltsc2022 base images (they must match the windows-2022 runner kernel); Nano Server no longer ships PowerShell, so use the PowerShell team image.
@mlocati
mlocati force-pushed the docker-tests-in-github-actions branch from b8fc2db to fd67de4 Compare October 28, 2026 14:37
Get-ComputerInfo is not available on Nano Server; setup.ps1 and the Docker test step must fail when Pester can't be installed or loaded, instead of silently passing.
Edit-FolderInPath compared an empty string against $null, so it tried to set an empty PATH, which Set-EnvVar rejected. This made Uninstall-Php and Remove-PhpSwitcher fail when the user or system PATH variable does not exist (eg in Nano Server containers).
@mlocati
mlocati merged commit caaf1a6 into master Oct 28, 2026
4 checks passed
@mlocati
mlocati deleted the docker-tests-in-github-actions branch October 28, 2026 15:08
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.

1 participant