Fix: Change Execute method to protected override in all Command classes#171
Merged
Conversation
The Spectre.Console.Cli 0.55.0 update changed the base Command.Execute method from public to protected. This updates all command implementations to use protected override instead of public override. Fixes compilation errors in: - AppCommand.cs - BenchmarkCommand.cs - InfoCommand.cs - ListCommand.cs - WorkflowCommand.cs
… as public override - Bump Spectre.Console.Cli from 0.53.1 to 0.55.0 in Directory.Packages.props - Keep Execute methods as public override (base class uses public, not protected) - All command classes now compatible with Spectre.Console 0.55.2 and Spectre.Console.Cli 0.55.0
….Cli 0.55.0 With Spectre.Console.Cli 0.55.0, the base Command.Execute method is now protected, not public. Updated all command classes to use protected override: - AppCommand.cs - BenchmarkCommand.cs - InfoCommand.cs - ListCommand.cs - WorkflowCommand.cs
Fixed whitespace formatting issue that was causing dotnet format check to fail.
…ifiers - Reverted all files to use CRLF line endings (\r\n) with UTF-8 BOM - Changed Execute methods from public to protected override (required for Spectre.Console.Cli 0.55.0) - Updated Spectre.Console.Cli version to 0.55.0 - Removed double semicolon from WorkflowCommand.cs line 66 This ensures minimal code diff focused only on the necessary changes.
- Changed to pure CRLF line endings without UTF-8 BOM - Maintains the access modifier changes (protected override) - Fixes the CS1003 and CS1002 syntax errors from the previous commit
…age versions - Removed all BOM and CRLF line ending changes - Using original LF line endings from main branch - Changed Execute methods to protected override for Spectre.Console.Cli 0.55.0 - Updated Spectre.Console.Cli version from 0.53.1 to 0.55.0 in Directory.Packages.props - Minimal diff focused only on necessary code changes
… modifiers - Restored original CRLF line endings (\r\n) and UTF-8 BOM from main branch - Only changed Execute methods from public to protected override - Removed double semicolon from WorkflowCommand.cs line 66 - Minimal functional code changes while preserving original file encoding
- Changed Execute methods from public to protected override for Spectre.Console.Cli 0.55.0 - Preserved exact file encoding with UTF-8 BOM and CRLF line endings - Minimal code changes focused only on access modifier update
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
1 participant
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.
The Spectre.Console.Cli 0.55.0 update changed the base Command.Execute method from public to protected. This updates all command implementations to use protected override instead of public override.
Fixes compilation errors in: