Skip to content
This repository was archived by the owner on Jan 23, 2026. It is now read-only.

Fix PowerShell installation issues in install.ps1#257

Merged
jhaynie merged 12 commits intomainfrom
devin/1746052783-fix-powershell-install
May 1, 2025
Merged

Fix PowerShell installation issues in install.ps1#257
jhaynie merged 12 commits intomainfrom
devin/1746052783-fix-powershell-install

Conversation

@devin-ai-integration
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot commented Apr 30, 2025

Fix PowerShell Installation Issues in install.ps1

Changes Made

  • Fixed PowerShell variable references with proper ${} syntax for colon handling
  • Improved error handling in catch blocks using local variables for error messages
  • Added verification that command is accessible after installation
  • Added clear instructions for manual setup if needed

Testing

  • All CI tests now pass, including the Windows PowerShell install script test
  • Verified the script works on Windows Server 2022 in CI environment

Link to Devin run

https://app.devin.ai/sessions/88d8764a13934b0289fd3f8422cd2766

Requested by: jhaynie@agentuity.com

- Improve PowerShell completion setup with better error handling
- Add temporary function to make command available in current session
- Provide clear instructions for manual setup if needed
- Fix issue where CLI is not recognized after installation

Co-Authored-By: jhaynie@agentuity.com <jhaynie@gmail.com>
@devin-ai-integration devin-ai-integration Bot requested a review from jhaynie April 30, 2025 22:41
@devin-ai-integration
Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

devin-ai-integration Bot and others added 11 commits April 30, 2025 22:48
…locks

- Properly escape the automatic variable using $(install.ps1) syntax
- Addresses CI failure in Windows PowerShell install script test

Co-Authored-By: jhaynie@agentuity.com <jhaynie@gmail.com>
…locks

- Properly escape the automatic variable using ${_} syntax instead of $(install.ps1)
- Addresses CI failure in Windows PowerShell install script test

Co-Authored-By: jhaynie@agentuity.com <jhaynie@gmail.com>
…rors

- Simplify error messages to ensure compatibility with Windows PowerShell
- Addresses CI failure in Windows PowerShell install script test

Co-Authored-By: jhaynie@agentuity.com <jhaynie@gmail.com>
- Replace  with  for better compatibility
- Addresses CI failure in Windows PowerShell install script test

Co-Authored-By: jhaynie@agentuity.com <jhaynie@gmail.com>
…andling

Co-Authored-By: jhaynie@agentuity.com <jhaynie@gmail.com>
@jhaynie jhaynie merged commit 3fbd4dc into main May 1, 2025
12 checks passed
@jhaynie jhaynie deleted the devin/1746052783-fix-powershell-install branch May 1, 2025 00:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant