Skip to content

Remove unused execution_rpc CLI parameter from opstack server#711

Open
sashaodessa wants to merge 1 commit intoa16z:masterfrom
sashaodessa:patch-2
Open

Remove unused execution_rpc CLI parameter from opstack server#711
sashaodessa wants to merge 1 commit intoa16z:masterfrom
sashaodessa:patch-2

Conversation

@sashaodessa
Copy link
Copy Markdown

PR Description

Removes unused execution_rpc CLI parameter from the opstack server binary that was declared but never used.

Problem

The execution_rpc field in the Cli struct was defined as a required CLI argument but was never referenced in the code. This created a misleading interface where users could pass a flag that would be silently ignored.

Solution

  • Removed the unused execution_rpc: Url field from the Cli struct
  • CLI interface now accurately reflects the actual functionality

Impact

  • Improves CLI usability by removing confusing unused parameters
  • Reduces technical debt
  • No breaking changes (parameter was unused)

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.

1 participant