Update EIP-658: fix grammar #17254
Annotations
8 errors and 2 warnings
EIPS/eip-658.md#L16
error[markdown-re-eip-dash]: proposals must be referenced with the form `EIP-N` (not `EIPN` or `EIP N`)
--> EIPS/eip-658.md
|
16 | With the introduction of the REVERT opcode in EIP140, it is no longer possible for users to assume that a transaction failed if it consumed all gas. As a result, there is no clear mechanism for callers to determine whether a transaction succeeded and the state changes contained in it were applied.
|
= info: the pattern in question: `(?i)eip[\s]*[0-9]+`
= help: see https://ethereum.github.io/eipw/markdown-re-eip-dash/
|
EIPS/eip-658.md#L20
error[markdown-re-eip-dash]: proposals must be referenced with the form `EIP-N` (not `EIPN` or `EIP N`)
--> EIPS/eip-658.md
|
20 | Instead, we propose to replace the intermediate state root, already obsoleted by EIP98, with the return status (1 for success, 0 for failure). This both allows callers to determine success status, and remedies the previous omission of return data from the receipt.
|
= info: the pattern in question: `(?i)eip[\s]*[0-9]+`
|
EIPS/eip-658.md#L11
error[markdown-req-section]: body is missing section(s): `Security Considerations`
--> EIPS/eip-658.md
|
|
= help: must be at the second level (`## Heading`)
= help: see https://ethereum.github.io/eipw/markdown-req-section/
|
EIPS/eip-658.md#L4
error[preamble-author]: preamble header `author` must contain at least one GitHub username
--> EIPS/eip-658.md
|
4 | author: Nick Johnson <[email protected]>
|
= help: see https://ethereum.github.io/eipw/preamble-author/
|
EIPS/eip-658.md#L3
error[preamble-len-title]: preamble header `title` value is too long (max 44)
--> EIPS/eip-658.md:3:7
|
3 | title: Embedding transaction status code in receipts
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ too long
|
= help: see https://ethereum.github.io/eipw/preamble-len-title/
|
EIPS/eip-658.md#L5
error[preamble-order]: preamble header `type` is out of order
--> EIPS/eip-658.md
|
5 | type: Standards Track
|
= help: `type` should come after `status`
= help: see https://ethereum.github.io/eipw/preamble-order/
|
EIPS/eip-658.md#L1
error[preamble-req]: preamble is missing header(s): `description`, `discussions-to`
--> EIPS/eip-658.md
|
|
= help: see https://ethereum.github.io/eipw/preamble-req/
|
|
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@47fbe2df0ad0e27efb67a70beac3555f192b062f. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
The following artifacts were uploaded using a version of actions/upload-artifact that is scheduled for deprecation: "pr_number".
Please update your workflow to use v4 of the artifact actions.
Learn more: https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/
|
The logs for this run have expired and are no longer available.
Loading