fix: #455 #456
fix: #455 #456
Conversation
There was a problem hiding this comment.
Pull request overview
This PR fixes issue #455, which was a critical bug where the function was returning an undefined variable. The main fix involves correcting the return statement (from $oaUnit to proper pipeline output), and additionally enhances the function with pipeline support and improved code readability.
Key changes:
- Bug fix: Removed incorrect
Return $oaUnitstatement that referenced a non-existent variable (the variable was actually$oaUnits) - Pipeline support: Added
ValueFromPipeline = $trueparameter attribute and wrapped logic in aprocessblock - Code refactoring: Replaced manual array accumulation with
ForEach-Objectpipeline pattern for cleaner, more PowerShell-idiomatic code
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Modules/Capa.PowerShell.Module.SDK.Unit/Dev/Get-CapaUnitRelations.ps1
Outdated
Show resolved
Hide resolved
…ons.ps1 Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
🎉 This PR is included in version 1.15.0-Prerelease.8 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 1.15.1-Prerelease.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 1.15.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
No description provided.