From 57b1b97c3db63dc084c7851876c5b02a91bce44b Mon Sep 17 00:00:00 2001 From: Claude Date: Thu, 1 Jan 2026 15:56:39 +0000 Subject: [PATCH] docs: fix grammar in execute plugin files - Fix "using in live networks" -> "on live networks" (execute.py:2) - Fix missing spaces between concatenated help strings in execute.py - Fix "using for test filling" -> "used for test filling" (pre_alloc.py:1) - Fix missing spaces in concatenated help strings in remote.py --- .../cli/pytest_commands/plugins/execute/execute.py | 8 ++++---- .../cli/pytest_commands/plugins/execute/pre_alloc.py | 2 +- .../cli/pytest_commands/plugins/execute/rpc/remote.py | 6 +++--- repo_subpaths_for_grammar_check.md | 2 +- 4 files changed, 9 insertions(+), 9 deletions(-) diff --git a/packages/testing/src/execution_testing/cli/pytest_commands/plugins/execute/execute.py b/packages/testing/src/execution_testing/cli/pytest_commands/plugins/execute/execute.py index 93b84b8065..1e2a9fc43c 100644 --- a/packages/testing/src/execution_testing/cli/pytest_commands/plugins/execute/execute.py +++ b/packages/testing/src/execution_testing/cli/pytest_commands/plugins/execute/execute.py @@ -1,5 +1,5 @@ """ -Test execution plugin for pytest, to run Ethereum tests using in live networks. +Test execution plugin for pytest, to run Ethereum tests on live networks. """ import os @@ -74,7 +74,7 @@ def pytest_addoption(parser: pytest.Parser) -> None: default=None, help=( "Default max priority fee per gas used for transactions, " - "unless overridden by the test." + "unless overridden by the test. " "Default=None (1.5x current network max priority fee per gas)" ), ) @@ -85,7 +85,7 @@ def pytest_addoption(parser: pytest.Parser) -> None: type=int, default=None, help=( - "Default max fee per blob gas used for transactions, unless overridden by the test." + "Default max fee per blob gas used for transactions, unless overridden by the test. " "Default=None (1.5x current network max fee per blob gas)" ), ) @@ -97,7 +97,7 @@ def pytest_addoption(parser: pytest.Parser) -> None: type=int, help=( "Maximum gas used to execute a single transaction. " - "Will be used as ceiling for tests that attempt to consume the entire block gas limit." + "Will be used as ceiling for tests that attempt to consume the entire block gas limit. " f"(Default: {EnvironmentDefaults.gas_limit // 4})" ), ) diff --git a/packages/testing/src/execution_testing/cli/pytest_commands/plugins/execute/pre_alloc.py b/packages/testing/src/execution_testing/cli/pytest_commands/plugins/execute/pre_alloc.py index 6127453c86..50372181b7 100644 --- a/packages/testing/src/execution_testing/cli/pytest_commands/plugins/execute/pre_alloc.py +++ b/packages/testing/src/execution_testing/cli/pytest_commands/plugins/execute/pre_alloc.py @@ -1,4 +1,4 @@ -"""Pre-allocation fixtures using for test filling.""" +"""Pre-allocation fixtures used for test filling.""" from itertools import count from pathlib import Path diff --git a/packages/testing/src/execution_testing/cli/pytest_commands/plugins/execute/rpc/remote.py b/packages/testing/src/execution_testing/cli/pytest_commands/plugins/execute/rpc/remote.py index 423697aa65..68bffd9a0f 100644 --- a/packages/testing/src/execution_testing/cli/pytest_commands/plugins/execute/rpc/remote.py +++ b/packages/testing/src/execution_testing/cli/pytest_commands/plugins/execute/rpc/remote.py @@ -54,8 +54,8 @@ def pytest_addoption(parser: pytest.Parser) -> None: default=None, dest="engine_endpoint", help="Engine endpoint to an execution client, which implies that the execute command " - "will be used to drive the chain. If not provided, it's assumed that the execution client" - "is connected to a beacon node and the chain progresses automatically. If provided, the" + "will be used to drive the chain. If not provided, it's assumed that the execution client " + "is connected to a beacon node and the chain progresses automatically. If provided, the " "JWT secret must be provided as well.", ) engine_rpc_group.addoption( @@ -73,7 +73,7 @@ def pytest_addoption(parser: pytest.Parser) -> None: action="store", default=None, dest="engine_jwt_secret_file", - help="Path to a file containing the JWT secret to be used to authenticate with the engine" + help="Path to a file containing the JWT secret to be used to authenticate with the engine " "endpoint. The file must contain only the JWT secret as a hex string.", ) diff --git a/repo_subpaths_for_grammar_check.md b/repo_subpaths_for_grammar_check.md index 0b338956af..8d808015d8 100644 --- a/repo_subpaths_for_grammar_check.md +++ b/repo_subpaths_for_grammar_check.md @@ -8,7 +8,7 @@ Status legend: `[ ]` pending, `[~]` in progress, `[x]` complete - [x] `packages/testing/src/execution_testing/cli/pytest_commands/plugins/consume/**/*.py` (~32 files) - [x] `packages/testing/src/execution_testing/cli/pytest_commands/plugins/filler/**/*.py` (~27 files) -- [ ] `packages/testing/src/execution_testing/cli/pytest_commands/plugins/execute/**/*.py` (~21 files) +- [x] `packages/testing/src/execution_testing/cli/pytest_commands/plugins/execute/**/*.py` (~21 files) - [ ] `packages/testing/src/execution_testing/cli/pytest_commands/plugins/forks/**/*.py`, `shared/**/*.py`, `help/**/*.py` (~16 files) - [ ] `packages/testing/src/execution_testing/cli/pytest_commands/plugins/*.py` (root plugins, ~6 files)