Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add EIP: Versioning Scheme for EIPs #8034

Merged
merged 23 commits into from
Jun 5, 2024
Merged
Changes from 15 commits
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
8ac6e54
Initial versioning scheme draft with incomplete case study
danceratopz Nov 1, 2023
4dbe8bb
Improvements and add a versioning + testing toolchain flowchart
danceratopz Nov 1, 2023
56e1f34
More improvements
danceratopz Nov 1, 2023
133ad54
More precise phrasing for abstract
danceratopz Nov 3, 2023
3cab575
Remove mermaid flow diagram and other clean-up
danceratopz Dec 8, 2023
3795273
First version fo the 4788 versioning vase study
danceratopz Dec 8, 2023
4a85fc0
Clean-up and remove template comments
danceratopz Dec 13, 2023
a2b6c43
Add eth magicians discussions link
danceratopz Dec 13, 2023
1d5a31a
Update eip-versioning-scheme-for-eips.md
smartprogrammer93 Dec 13, 2023
c8b32fa
remove not needed sections
smartprogrammer93 Dec 13, 2023
bcc524d
add eip to be assigned so a suggestion can easily be made
smartprogrammer93 Dec 13, 2023
564d13c
Add assigned EIP number to header.
danceratopz Dec 14, 2023
00a28b6
Rename eip with assigned number
danceratopz Dec 14, 2023
2a52b0e
Remove links to github.
danceratopz Dec 14, 2023
acae175
Merge branch 'master' into versioning-scheme-for-eips
smartprogrammer93 Jan 3, 2024
2a0eb65
remove requirement to include a github pr link in changelog entries
danceratopz Mar 13, 2024
0503e84
remove motivational content from rationale section
danceratopz Mar 13, 2024
8319b2d
add sem ver 2.0.0 doc to assets and link to it from eip text
danceratopz Mar 13, 2024
02769ff
simplify explanation of minor version
danceratopz Mar 14, 2024
91e978f
specify repo tooling mechanism to assist automatic version bumping
danceratopz Mar 14, 2024
494400f
Apply suggestions from code review
danceratopz Mar 14, 2024
7fae3bb
Merge branch 'versioning-scheme-for-eips' of github.com:danceratopz/E…
danceratopz Mar 14, 2024
92fb8ba
remove link to ethereum/execution-spec-tests
danceratopz Mar 14, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
107 changes: 107 additions & 0 deletions EIPS/eip-7577.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,107 @@
---
eip: 7577
title: Versioning Scheme for EIPs
description: Use a versioning scheme for EIPs based on changes made to their Specification section.
author: danceratopz (@danceratopz), Ahmad Bitar (@smartprogrammer93)
discussions-to: https://ethereum-magicians.org/t/add-eip-versioning-scheme-for-eips/17295
status: Draft
type: Meta
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if this shall be Informational or Meta. I like the idea and direction. I don't think every EIP needs adopt it

created: 2023-12-13
---

## Abstract

This EIP introduces a versioning scheme for [Standards Track](./eip-1.md#eip-types) EIPs by applying semantic versioning based on changes made to the EIP's Specification section once its status has changed from `Draft` to `Review`.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Might be a good idea to provide more in depth information about semantic versioning for people who aren't familiar. You could do something similar to: https://eips.ethereum.org/EIPS/eip-5139#versioning

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, I didn't know I could add an entire external document as an asset. Added in 8319b2d.


## Motivation

EIP specifications often receive increasing modifications as more people review them, which is generally the case as client teams start implementing the specifications and the community gains a better understanding of their interaction with the rest of the protocol. These changes can be difficult to track. In particular, as EVM reference tests are often not maintained (and generally not released) by client teams or the EIP's authors, it can be difficult to ascertain whether a release of reference tests (for example from ethereum/tests or ethereum/execution-spec-tests) is sufficient, or even valid, to test the latest version of an EIP's specifications or the specification as currently implemented by a client.
danceratopz marked this conversation as resolved.
Show resolved Hide resolved

This EIP proposes a semantic versioning scheme and an addition of a CHANGELOG section for EIPs that allows the community to easily track changes, their impact on current client and test implementations and maintain a clear history.

## Specification

danceratopz marked this conversation as resolved.
Show resolved Hide resolved
Once an EIP has moved out of "Draft" status, it MUST use the EIP versioning scheme outlined below. It MAY already use the versioning scheme in "Draft" status, which could be useful if the specification is actively being implemented. If more than one team is implementing the specification, it is RECOMMENDED to change the EIP's status to "Review".

The EIP versioning scheme MUST follow a semantic versioning scheme of `MAJOR.MINOR.PATCH`, which is applied as follows:

1. `MAJOR`: A breaking change to the specifications that requires an implementation change and a change to the reference tests.
2. `MINOR`: An addition to the specifications that does not require changing an existing implementation, but requires additional implementation and additional test coverage to be added to the reference tests.
3. `PATCH`: Any cosmetic change to, or a reformulation of, the EIP without specification change.

Before the EIP has moved out of Draft status and is being versioned, the version number MUST initially have `MAJOR` version `0`.

For every change made to an EIP via a PR made to ethereum/EIPs, the PR author MUST add a new entry to the CHANGELOG section of the EIP that outlines the changes made within the PR. This CHANGELOG entry MUST include the following:
danceratopz marked this conversation as resolved.
Show resolved Hide resolved

1. A new version number that follows the semantic versioning scheme outlined above.
2. The date when the changes where introduced.
3. A link to the ethereum/EIPs PR that implements the changes.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The current rules forbid links to PRs. This isn't a formatting issue, so you'll have to talk to EIP editors if you want this change.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed link requirement in 2a0eb65.

I think a link to the changes would be helpful to readers - I'll follow up with EIP editors to gauge interest.

4. A line for each change made to the EIP's specifications that includes a short description of the change.

Additionally, the new version MUST be added to the metadata header of the EIP's markdown file (to a new "version" field), so that it may be easily parsed.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

to a new "version" field

This should have higher billing. Like the first line after the RFCs. As well as the new changelog section.

A new header field `version` is added to new EIPs, as defined below, identifying the  draft and/or review version of the specification.

A new body section "Changelog" is added to new EIPs, as defined below, tracking the changes between draft and/or review versions of the specificaiton.

Also, we should update the EIP template to add these two new feaures as part of this EIP when it goes final.


Tooling SHOULD be added to the ethereum/EIPs repository that checks that the CHANGELOG is updated and that the version number in the metadata header matches the latest version number in the CHANGELOG.

EIPS that have dependencies on other EIPS SHOULD include the version of the dependency in the metadata "requires" field.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please provide more information about how this would look. EIP should include enough detail to be implementable.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

91e978f:

  • Removed the following requirement, as it's impractical to update all downstream EIPs upon a version change:

    EIPS that have dependencies on other EIPS SHOULD include the version of the dependency in the metadata "requires" field.

  • Modified the Specification with a suggestion for the mechanism to update an EIP version using repo tooling.


## Rationale

A semantic versioning scheme enables clearer communication within the community about EIP status and allows the impact of a change to be ascertained at first glance.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This belongs in the Motivation section.

The Motivation should be why is this proposal a good idea, where the Rationale should be why you made those specific decisions within the EIP.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed from Rational in 0503e84.


Making the version available in the EIP's metadata header additionally allows for programmatic parsing of the version number by tooling used in reference tests or by client teams. Currently, the ethereum/execution-spec-tests repository implements a rudimentary EIP version checker: EIP spec tests are required to declare the EIP's markdown file digest SHA that the test implementation was based on. The current value of the digest SHA is then polled via the Github API to verify that no changes have occurred since the test implementation. While this provides a warning to test implementers that the EIP has changed, it is clearly of limited use.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please avoid external links (ethereum/execution-spec-tests).

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed in 92fb8ba. The repository is still referred to, I hope that's ok?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fyi links to execution-spec-tests were recently allowed!


A richer versioning scheme, as defined by this EIP, can provide a lot of value to the testing toolchain. Client teams can provide an interface that reports the EIP version currently implemented and reference tests can specify the version they implement in generated tests as metadata. This allows a test runner to mark tests to xfail (expectedly fail) and issue a warning if the `MAJOR` or `MINOR` versions don't match. It would even be possible to automatically select the correct version of the reference tests to run against a client implementation, although given the pace of Ethereum development, it will likely be impractical to maintain and track multiple versions of tests.

### Case Study

This section explores how the versioning scheme would be applied to an existing EIPs recently under active development at the time of writing as an example.

The history of [EIP-4788](./eip-4788.md) contains many changes to its specification. EIP-4788 was updated to status "Review" on 2023-11-28 in #7992. This case study assumes, however, that the EIP moved to status "Review" as of #6859 and updated to version 1.0.0 due to the start of a client team implementation.
danceratopz marked this conversation as resolved.
Show resolved Hide resolved

#### Changelog

- 9.0.1 - 2023-09-26: Update ring buffer size rationale for new ring buffer size, #7786.
- 9.0.0 - 2023-09-26: Post audit tweaks, #7672.
- Verify timestamp is non-zero.
- Make `HISTORY_BUFFER_LENGTH` prime (8191).
- Load calldata once.
- Update `BEACON_ROOTS_ADDRESS`.
- 8.0.1 - 2023-08-28: 4788 cleanups, #7532.
- 8.0.0 - 2023-08-24: Initial stab at v2, #7456.
- Require timestamp input to be exactly 32 bytes.
- Revert if timestamp input does not match stored value (instead of returning zeroed word).
- Remove precompile concept, use regular smart contract with provided bytecode.
- 7.0.3 - 2023-08-01: Mention genesis block with no existing beacon block root case, #7445.
- 7.0.2 - 2023-07-07: Explicitly specify header schema, #7297.
- 7.0.1 - 2023-07-07: Fix typo, #7293.
- 7.0.0 - 2023-07-05: Bound precompile storage, #7178.
- 6.0.1 - 2023-06-13: Clarify header and validity sections, #7179.
- 6.0.0 - 2023-06-12: Update precompile address, #7173.
- 5.0.0 - 2023-05-31: Key beacon roots by root, #7107.
- 4.0.0 - 2023-05-24: Favor stateful precompile over opcode, #7065.
- 3.0.0 - 2023-05-17: Send current slot from CL to avoid timestamp conversions, #7037.
- 2.0.1 - 2023-05-15: Fix typo, #7005.
- 2.0.0 - 2023-05-03: Update opcode to avoid clash, #6980.
- 1.0.1 - 2023-04-13: Minor nits, #6870.
- 1.0.0 - 2023-04-11: Use block roots; update to status "Draft", #6859.
- Update to "Draft" due to client implementation (NethermindEth/nethermind#5476).
- Use block roots instead of state roots.
- Roots are stored keyed by slot.
- Use of ring buffer in state.
- Use header timestamps to derive slot numbers, rather than consume additional header space.
- 0.2.1 - 2023-02-04: Update to status "Stagnant", #6432.
- 0.2.0 - 2022-06-29: Rename "beacon block root" to "beacon state root", #5090.
- 0.1.1 - 2022-05-06: Force usage of included LICENSE file, #5055.
- 0.1.0 - 2022-02-17: Add EIP-4788: Beacon state root in EVM, #4788.

## Backwards Compatibility

It is not necessary to retroactively add a CHANGELOG or versions for versions of the EIP prior to the introduction of this EIP. Upon the next change to the EIP's Specification section, the author MUST introduce a CHANGELOG section and a version number that follows the semantic versioning scheme outlined above.

## Security Considerations

None.

## Copyright

Copyright and related rights waived via [CC0](../LICENSE.md).
Loading