Skip to content

ci: replace Windows cmd shell with PowerShell - #4812

Merged
lidavidm merged 1 commit into
apache:mainfrom
efegokdemir:codex/issue-4269-remove-cmd-shell
Sep 28, 2026
Merged

lidavidm merged 1 commit into
apache:mainfrom
efegokdemir:codex/issue-4269-remove-cmd-shell

Conversation

@efegokdemir

@efegokdemir efegokdemir commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

What's Changed

Replace the Windows driver-manager workflow step's cmd.exe shell with PowerShell.

Closes #4269.

AI assistance was used during development; the complete diff and test results were reviewed by the author.

Signed-off-by: Efe Gökdemir <efe@rexcode.co.uk>
@lidavidm lidavidm changed the title ci(java): replace Windows cmd shell with PowerShell ci: replace Windows cmd shell with PowerShell Sep 28, 2026
@lidavidm
lidavidm requested a balanced review from Copilot September 28, 2026 03:51

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟢 Approval recommended

The PowerShell implementation preserves existing behavior and removes the remaining workflow cmd.exe usage.

Review effort: Balanced
Findings: None

What changed in this PR

Replaces the Windows driver-manager CI step’s cmd.exe usage with equivalent PowerShell commands, resolving #4269.

Changes:

  • Constructs build paths using PowerShell.
  • Preserves CMake configuration, build, installation, and error handling.
File Description
.github/​workflows/​java.yml Migrates the Windows driver-manager build step to PowerShell.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@lidavidm
lidavidm merged commit f1378d6 into apache:main Sep 28, 2026
20 of 22 checks passed
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.

ci: eliminate use of cmd in workflows

3 participants