yotsuda
yotsuda
When displaying multiple progress bars, the screen becomes significantly cluttered. This Copy-Item cmdlet in my provider displays multiple progress bars to represent the sub-progress of the contents being copied in...
@SteveL-MSFT Thank you so much for taking quick action!
Hello @SteveL-MSFT , could you please let me know if this issue has been resolved? Thank you.
@iSazonov @MatejKafka Thanks for the feedback! I've updated `ToRegex()` to return `Regex` instead of `string`. The implementation now caches the `Regex` instance for subsequent calls. Changes (ce1dcf368): - `ToRegex()` returns...
I attempted to implement the fix as WG proposed - modifying the `SetDriveScriptBlock` (embedded functions like `function:C:`). Here is the diff I tried: ```diff diff --git a/src/System.Management.Automation/engine/InitialSessionState.cs b/src/System.Management.Automation/engine/InitialSessionState.cs index 62308282d..b6d7092e4...
@krzydoug Thank you for looking at this! @MHughe57 Exactly right. The [current validation](https://github.com/PowerShell/PowerShell/blob/master/src/Microsoft.PowerShell.Commands.Utility/commands/utility/CsvCommands.cs#L123) checks if both parameters are **bound**, regardless of their values: ```csharp if (this.MyInvocation.BoundParameters.ContainsKey(nameof(IncludeTypeInformation)) && this.MyInvocation.BoundParameters.ContainsKey(nameof(NoTypeInformation))) ``` **Proposed...
> @yotsuda Please look test errors. Fixed in ce12d7790. The original implementation using `IsWriteVerboseEnabled()` was affected by `-Debug` parameter, causing unintended behavior. Created a dedicated method `ShouldEmitVerboseFromShouldProcess()` that checks only:...
> Hmm, `IsWriteVerboseEnabled` is not directly check Debug parameter. So it is not right fix. @iSazonov You're right that `IsWriteVerboseEnabled()` doesn't directly check `-Debug`. However, it uses the `VerbosePreference` property...
@MartinGC94 Thank you for the insightful review! You're absolutely right - the `[hidden()]` syntax wasn't available for PowerShell classes (missing type accelerator), and there were no tests for the AST-based...