build: rename tests/ directory to lowercase for CI portability - #496
Merged
Merged
Conversation
Delete the old Azure Pipelines / GitHub Actions build pipeline and its supporting scripts, now that psake + PowerShellBuild (#493) is the build entry point: - .github/workflows/github.yml (bespoke GitHub Actions CI) - azure-pipelines.yml and templates/ (Azure Pipelines templates) - Build/ (Build-Module.ps1, Initialize-Environment.ps1, Invoke-ModuleTests.ps1, New-Changelog.ps1, Register-FileSystemRepository.ps1) - Deploy/ (PSDeploy scripts; only caller was the removed workflow) Also drops the now-dead Azure Pipelines build/coverage badges and the dev.azure.com build-artifact download link from README.md. Closes #486
Windows/macOS filesystems are case-insensitive; Linux is not. The PowerShellOrg reusable CI workflow (powershell-ci.yml) uploads test results from a hardcoded ./tests/out/testResults.xml path, which would silently fail to find artifacts on the ubuntu-latest runner while the directory was capitalized Tests/. Rename Tests/ -> tests/ and update the two path references: - .gitignore (tests/out/**) - psakeFile.ps1 ($PSBPreference.Test.RootDir) Prep for #488.
HeyItsGilbert
requested review from
steviecoaster and
vexx32
as code owners
September 18, 2026 00:13
This was referenced Sep 18, 2026
There was a problem hiding this comment.
🟢 Approval recommended
The directory rename and both path references are consistent, with no unresolved issues found.
Pull request overview
Renames the test directory to lowercase so Pester and CI artifact paths work on case-sensitive filesystems.
Changes:
- Renamed
Tests/totests/. - Updated Pester’s test root and
.gitignoreoutput path. - Preserved all test contents.
File summaries
| File | Description |
|---|---|
.gitignore |
Ignores lowercase test output. |
psakeFile.ps1 |
Uses the lowercase test root. |
tests/* |
Case-only renames of the test suite files. |
Review details
- Files reviewed: 1/29 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Stacked on #495.
Windows/macOS filesystems are case-insensitive; Linux is not. The PowerShellOrg reusable CI workflow (
powershell-ci.yml) uploads test results from a hardcoded./tests/out/testResults.xmlpath, which would silently fail to find artifacts on theubuntu-latestrunner while the directory stayed capitalizedTests/.Renames
Tests/->tests/and updates the two path references (.gitignore,psakeFile.ps1).Prep for #488.
Stack: #495 -> 2/3 -> #497